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

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants