Skip to content

Set correct media type#62

Merged
FlipperPA merged 1 commit intomainfrom
willtho89-correct-media-type
Sep 30, 2022
Merged

Set correct media type#62
FlipperPA merged 1 commit intomainfrom
willtho89-correct-media-type

Conversation

@willtho89
Copy link
Copy Markdown
Contributor

according to https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/MIME_types/Common_types application/vnd.openxmlformats-officedocument.spreadsheetml.sheet should be used. Closes #60

@willtho89
Copy link
Copy Markdown
Contributor Author

@FlipperPA is there a reason application/xlsx was used?

@FlipperPA
Copy link
Copy Markdown
Member

@willtho89 Not in particular, just from years of habit dating back to my PHP days. 👍 This sounds right to me, let's just make sure we still have it properly open the default spreadsheet app with the new media_type.

@FlipperPA FlipperPA merged commit cda74db into main Sep 30, 2022
@FlipperPA
Copy link
Copy Markdown
Member

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

Successfully merging this pull request may close these issues.

Support Common media_type

2 participants