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

.CSV import fails if there is a description row #970

Closed
luke-quadratic opened this issue Jan 8, 2024 · 4 comments
Closed

.CSV import fails if there is a description row #970

luke-quadratic opened this issue Jan 8, 2024 · 4 comments
Labels
type: bug something is broken type: papercut annoying but not the end of the world

Comments

@luke-quadratic
Copy link
Contributor

It's not uncommon for .CSV format to have a row that is a description right after the sheet title. In this case the .CSV import is getting cut off. E.g. the following .CSV should show up as column names and data, instead it will just import as the column names and the description (date range in this case).
CleanShot 2024-01-08 at 13 25 00@2x

@ddimaria
Copy link
Collaborator

ddimaria commented Jan 8, 2024

Without an import UI, I don't think we can properly handle this case. If there was an import UI, we could ask for the row number to start importing from, which could address this issue.

@luke-quadratic
Copy link
Contributor Author

Noted, popping down to low prio in that case since the return on time seems relatively low. @ddimaria

@davidkircos
Copy link
Collaborator

I think it should just import the description into a cell on the second line. Then the data starting the row before that. No need for a import UI

@luke-quadratic
Copy link
Contributor Author

reopen if any users run into issues around edge case CSVs, original case appears fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug something is broken type: papercut annoying but not the end of the world
Projects
Archived in project
Development

No branches or pull requests

3 participants