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

Add support for CSV files #15

Open
patcon opened this issue Nov 3, 2018 · 2 comments
Open

Add support for CSV files #15

patcon opened this issue Nov 3, 2018 · 2 comments

Comments

@patcon
Copy link

patcon commented Nov 3, 2018

Unless I'm misunderstanding, and TSV's are superior in some way :)

@kasbah
Copy link
Member

kasbah commented Nov 4, 2018

TSV are a bit easier to parse because you don't need to worry about quote marks around fields with commas in them. People generally don't put tabs into their fields. I think that's why it's used as the clipboard format for spreadsheets. That's why we use it.

We actually already have support for CSV (plus Excel and Libre/OpenOffice) in the parser library. It's already enabled on Kitspace and bom-builder.kitspace.org. We should pull that in and make use of it in the extension for the "blue button" loading thing.

@patcon
Copy link
Author

patcon commented Nov 4, 2018

Ah ok! Yeah, that was the only one I tried :) I'm happy to investigate when I get a sec

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

No branches or pull requests

2 participants