-
Notifications
You must be signed in to change notification settings - Fork 10.3k
fix(gatsby): remove default noscript tag from html.js template #20023
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
fix(gatsby): remove default noscript tag from html.js template #20023
Conversation
Hey, thanks for your first contribution! 🎉 It seems like our tests are failing. To fix this, could you run |
@wardpeet Thank you! I ran the command you said, let's hope it passes the tests 😅 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @gilbertoayala12! ❤️
Verified that lighthouse no longer list no content as an issue when JS is disabled with offline plugin installed, so this is good to go in!
Holy buckets, @gilbertoayala12 — we just merged your PR to Gatsby! 💪💜 Gatsby is built by awesome people like you. Let us say “thanks” in two ways:
If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’. Thanks again! |
@gilbertoayala @pieh @wardpeet I'm desperately trying to remove the I'm currently on Version I've tried a few other things, and haven't been successful. |
@nkhil This changes was added in |
@pieh Thank you so much - that's worked! |
Description
Removed banner saying
This app works best with JavaScript enabled
as requested.This is no longer necessary because
gatsby-plugin-offline
was fixed to refresh when javascript was disabled plus lighthouse checks if any content is available.Related Issues
Fixes #20021