Skip to content
New issue

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

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

Already on GitHub? Sign in 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
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants