Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Add CSV support to upload function #44

Open
lilchurro opened this issue Apr 24, 2018 · 2 comments
Open

Add CSV support to upload function #44

lilchurro opened this issue Apr 24, 2018 · 2 comments

Comments

@lilchurro
Copy link
Contributor

Allow user to upload a CSV file; convert it to json format and feed it to CRviz.

@lilchurro lilchurro self-assigned this Apr 24, 2018
@lilchurro
Copy link
Contributor Author

lilchurro commented May 21, 2018

Implemented in pull request #58

@lilchurro
Copy link
Contributor Author

As there are some issues (see: issue #60) with the current implementation, I'm reopening this enhancement ticket for more detailed planning. Currently, I had done some basic parsing and verification on an imported file, and then passed the contents on to JSON.parse if parsing as CSV didn't work out. The result was the user wouldn't really be notified about formatting errors from the CSV file, and complicated data types (i.e., quoted field values, in this case) weren't really being parsed properly.

All this to say, adding native CSV support is actually rather nontrivial, so I'm reworking the code to provide better support for this functionality.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants