Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Operation request: CSV to JSON #277

Closed
n1474335 opened this issue Mar 23, 2018 · 8 comments
Closed

Operation request: CSV to JSON #277

n1474335 opened this issue Mar 23, 2018 · 8 comments

Comments

@n1474335
Copy link
Member

n1474335 commented Mar 23, 2018

Summary

An operation to convert a CSV input to a JSON output or an array of arrays.

@Vimal-Raghubir
Copy link

Hi there. I believe I can implement this feature. Do you need the CSV to JSON logic just implemented or does it need to be attached to any portion of the code?

@mattnotmitt
Copy link
Collaborator

Follow this page for a guide on how to integrate the logic into the main interface.

@Vimal-Raghubir
Copy link

Ok thanks. And do you need it to handle CSV string or CSV files?

@mattnotmitt
Copy link
Collaborator

mattnotmitt commented Mar 25, 2018

I believe that as long as you use the string type for the operation input, when you drag in a CSV file it'll work as if you just copied the text.

@Vimal-Raghubir
Copy link

Ok thank you for the clarification. Then all I will need to handle is parsing the CSV input to either array or JSON. Is it possible if I can be assigned to this issue since I would like to make a PR?

@mattnotmitt
Copy link
Collaborator

We don't really assign to issues on this project. Just make a PR when you're ready, referencing this issue.

@Vimal-Raghubir
Copy link

Hi there. Following up to the CSV parser code, I was wondering if I should import my module to the Default or the OpModules.js since I am importing a npm library to do the parsing.

@Kladdkaka
Copy link

@Vimal-Raghubir Hi! May I ask why you decided to close your pull request?

BRAVO68WEB pushed a commit to BRAVO68WEB/CyberChef that referenced this issue May 29, 2022
…igurable

[FIX] Make routing mode user-configurable
Fixes gchq#274
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

4 participants