search

django-commons / drf-excel

An XLSX spreadsheet renderer for Django REST Framework.
BSD 3-Clause "New" or "Revised" License
217 stars 40 forks source link

Set correct media type #62

Closed willtho89 closed 2 years ago

willtho89 commented 2 years ago

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 commented 2 years ago

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

FlipperPA commented 2 years ago

@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 commented 2 years ago

Thank you! This has been published to PyPI at 2.2.0:

https://github.com/wharton/drf-excel/releases/tag/2.2.0

https://pypi.org/project/drf-excel/2.2.0/