Skip to content

fix(sw): Fix service worker app-shell route that binds to a non-existent /index.html - #1066

Closed
SharifIbrahimDev wants to merge 1 commit into
rinafcode:mainfrom
SharifIbrahimDev:fix/sw-app-shell-route
Closed

fix(sw): Fix service worker app-shell route that binds to a non-existent /index.html#1066
SharifIbrahimDev wants to merge 1 commit into
rinafcode:mainfrom
SharifIbrahimDev:fix/sw-app-shell-route

Conversation

@SharifIbrahimDev

Copy link
Copy Markdown

Closes #915

Summary of Changes

  • Replaced CRA legacy createHandlerBoundToURL('/index.html') binding in src/serviceWorker.ts with a Next.js App Router compatible NetworkFirst navigation handler.
  • Configured fallback to precached offline page when network is unavailable.
  • Created unit test suite in src/__tests__/serviceWorker.test.ts.

Technical Requirements Checklist

  • Remove non-existent /index.html app-shell binding.
  • Provide a Next.js App Router compatible navigation strategy.

@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@SharifIbrahimDev Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

Fix service worker app-shell route that binds to a non-existent /index.html

2 participants