Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented May 11, 2023

@danxuliu
Copy link
Member

Sorry, but I am not qualified to review this 🤷

@danxuliu danxuliu removed their request for review May 11, 2023 19:00
include mime.types;
types {
application/javascript js mjs;
text/javascript js mjs;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this seems to be the actual change? 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes but we need the other one as well to gzip compress it.

@susnux susnux force-pushed the fix/admin-nginx-js-mime branch from b19529b to e329828 Compare May 15, 2023 10:28
@susnux susnux requested a review from ChristophWurst May 15, 2023 10:28
@susnux susnux force-pushed the fix/admin-nginx-js-mime branch from e329828 to 1ef2afc Compare May 15, 2023 10:31
@susnux susnux merged commit 1280705 into master May 15, 2023
@susnux susnux deleted the fix/admin-nginx-js-mime branch May 15, 2023 12:17
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.

4 participants