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 MIME type for CSV files #5

Merged
merged 1 commit into from
Nov 10, 2017
Merged

Conversation

guyfawcus
Copy link
Contributor

Add MIME type for comma-separated value files and use the UTF-8 charset:

  • text/csv; charset=utf-8

At the moment, they appear as type:

  • application/octet-stream

...which is the default type for unknown files.

This should fix the CSV file created by archmap.
Currently, when you open it in a browser some of
the text is incorrectly formatted.

References:

Add MIME type for comma-separated value files and use the UTF-8 charset:
  "text/csv; charset=utf-8"

At the moment, they appear as type:
  "application/octet-stream"

...which is the default type for unknown files.

This should fix the CSV file created by archmap.
Currently, when you open it in a browser some of
the text is incorrectly formatted.

References:

* https://archwomen.org/media/archmap/archmap.csv
* https://www.iana.org/assignments/media-types/text/csv
@guyfawcus
Copy link
Contributor Author

I got confused with the plain text file here ( #6 ) and actually haven't run into any problems with the text formatting in the CSV file. Firefox and Chromium both just download the file and on my iPhone the table is displayed properly, without errors.

Can't hurt to have it reported as the right thing though 😉

@Lyfjaberg Lyfjaberg merged commit 0be4bd4 into archwomen:master Nov 10, 2017
@meskarune meskarune self-assigned this Apr 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants