-
Notifications
You must be signed in to change notification settings - Fork 30
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
Bad-precaching-response with themes/min.css when trying to use EUI next.js Starter for PWA #25
Comments
This is just a hunch, but it could be that the theme switching method is the cause. Namely, using |
Thanks for the suggestion @thompsongl - I tried this with this PR - https://github.com/hidesminimally/engram/pull/1 and am at the same place: I've also tried
Which I think is also a relevant error... But interestingly enough, when I deploy it to vercel: https://engram-k3raxj6v1-hidesminimally.vercel.app/ it does not have the sass module loading error. Just the same errors as the original post -- 404 for build-manifest.json (and min.css though I've told the caching to ignore those files for now), which prevent the app from becoming a PWA due to the 404 crashing the serviceworker.js |
Update: Although I'm not sure this is totally correct because I see this 404 on the vercel deployment: And yet it is able to become a PWA..... |
I'm using EUI's Next.js starter and am trying to make this into a PWA -- here is the repo. I haven't done anything crazy different from the EUI starter. Upon running
npm run build; npm run start
I getworkbox-030153e1.js:1 Uncaught (in promise) bad-precaching-response: bad-precaching-response :: [{"url":"http://localhost:3000/public/themes/eui_theme_amsterdam_dark.e7b2d22ec11f782218ad.min.css","status":404}
Which I think results the PWA audit in Lighthouse to fail:
I've tried excluding the
themes/
folder from the caching step but this hasn't worked.Any ideas?
Thanks in advanced!
The text was updated successfully, but these errors were encountered: