Skip to content

use correct MIME type for js-script file#2200

Merged
abidlabs merged 2 commits into
gradio-app:mainfrom
daspartho:correct-mime-type-js
Sep 7, 2022
Merged

use correct MIME type for js-script file#2200
abidlabs merged 2 commits into
gradio-app:mainfrom
daspartho:correct-mime-type-js

Conversation

@daspartho
Copy link
Copy Markdown
Contributor

@daspartho daspartho commented Sep 7, 2022

Fixes: #1203

This makes the server use the proper MIME type for the js-script file.

I made the changes as suggested in the issue. @abidlabs could you please review the changes i made?

@abidlabs
Copy link
Copy Markdown
Member

abidlabs commented Sep 7, 2022

Hi @daspartho this seems reasonable. Were you able to reproduce the original issue and can you confirm that this fixes it?

@daspartho
Copy link
Copy Markdown
Contributor Author

I reproduced the original issue and was able to resolve it with this.

@abidlabs
Copy link
Copy Markdown
Member

abidlabs commented Sep 7, 2022

Great, I haven't been able to test it since I can't reproduce the original issues, but the code changes look reasonable to me. Let me format it and then merge it in!

@abidlabs abidlabs merged commit 1615104 into gradio-app:main Sep 7, 2022
@daspartho daspartho deleted the correct-mime-type-js branch September 8, 2022 15:15
@daspartho
Copy link
Copy Markdown
Contributor Author

daspartho commented Sep 8, 2022

Thank you for your valuable input :)

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.

No Output In Browser

2 participants