-
-
Notifications
You must be signed in to change notification settings - Fork 27k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add explicit "Opting Out of Caching" header #2822
Conversation
@@ -1571,7 +1574,8 @@ have decided that you would like to disable them for all your existing users, | |||
you can swap out the call to `serviceWorkerRegistration.register()` in | |||
[`src/index.js`](src/index.js) with a call to `serviceWorkerRegistration.unregister()`. | |||
After the user visits a page that has `serviceWorkerRegistration.unregister()`, | |||
the service worker will be uninstalled. | |||
the service worker will be uninstalled. Note that depending on how `/service-worker.js` is served, | |||
it make take up to 24 hours for the cache to be invalidated. |
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.
Should be may instead of make
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.
Feel free to submit a PR. 😄
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.
So that it's harder to miss.