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 extra information for mime type "text/comma-separated-values" #4943

Closed
landreev opened this issue Aug 10, 2018 · 3 comments
Closed

Add extra information for mime type "text/comma-separated-values" #4943

landreev opened this issue Aug 10, 2018 · 3 comments

Comments

@landreev
Copy link
Contributor

This is a 1-2 issue.

Most CSV files are identified as "text/csv". But we also accept "text/comma-separated-values" (as an ingestable format).
However, we only keep a "pretty", display type for "text/csv". (the mime display type properties bundle needs to be updated).
Also, we don't have a file extension mapping for this type. (this means if you download the original, or uningest, the filename extension "csv" will be lost).

Alternatively we can change "text/comma-separated-values" to "text/csv" when saving the original file info in the db.

@mheppler
Copy link
Contributor

Updating this for @landreev, as text/comma-separated-values=Comma Separated Values was added to MimeTypeDisplay.properties in commit 5a19537 on May 2, 2019 as part of #2202, released in 4.15.

@pdurbin pdurbin added the Type: Suggestion an idea label Nov 16, 2023
@pdurbin
Copy link
Member

pdurbin commented Jun 3, 2024

In the context of Croissant, specifically comparing output from Kaggle vs (upcoming) output from Dataverse, @goeffthomas made this observation:

"I see an encodingFormat of text/comma-separated-values. Kind of curious about that since I think text/csv is more the MIME type that's on https://www.iana.org/assignments/media-types/media-types.xhtml and https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/MIME_types "

I think he's right. text/csv seem much more common than text/comma-separated-values (1 million vs. 35k hits on Google respectively). Plus the specs above and https://en.wikipedia.org/wiki/Comma-separated_values all seem to point to text/csv being the preferred type.

@cmbz
Copy link

cmbz commented Aug 20, 2024

To focus on the most important features and bugs, we are closing issues created before 2020 (version 5.0) that are not new feature requests with the label 'Type: Feature'.

If you created this issue and you feel the team should revisit this decision, please reopen the issue and leave a comment.

@cmbz cmbz closed this as completed Aug 20, 2024
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