Closed pi0 closed 3 years ago
Merging #417 (dcb519f) into master (be2bc76) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #417 +/- ##
=======================================
Coverage 85.28% 85.28%
=======================================
Files 9 9
Lines 367 367
Branches 122 122
=======================================
Hits 313 313
Misses 54 54
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update be2bc76...dcb519f. Read the comment docs.
When opening chromium devtools on first load, it makes a request that violates Request.cache spec: (chromium#823392)
Solution is to rewrite workbox request by detecting this condition and change cache option to
default
Since there might be more causes for uncaught errors (https://github.com/nuxt-community/pwa-module/issues/176#issuecomment-665722983) this PR is also adding more context to production errors with full context:
Before: (production - debug disabled):
After: (production)
For more context, one may still enable production debugging in
nuxt.config
:This PR resolves #176 with best efforts. IMPORTANT: Disabling
clientsClaim
as workaround with other defaults is NOT SAFE and may cause caching problems and if you was doing it before, should keep it with default value