-
Notifications
You must be signed in to change notification settings - Fork 100
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
Conflict with WordPress Updates screen #247
Comments
Interesting. Apparently that iframed URL is not in the admin? Apparently the service worker's navigation routing needs to exempt that URL from handling. |
It looks like And it doesn't seem to happen every time I install updates, but I definitely recall this happening multiple times now. |
Ok, actually the admin also serves an offline template when it detects the user is offline. The iframe probably had the offline template and the message said that the server was offline, not the client, right? That offline detection is will have to be improved. |
I don't recall the wording exactly, but that would explain it I think! |
Could it be that the error failed with an internal server error? If the service worker doesn't see pwa-wp/wp-includes/js/service-worker-navigation-routing.js Lines 45 to 47 in f6800eb
Can you replicate the issue by causing an error during update? |
I had a few available plugin updates on my site and went to Dashboard -> Updates to install them.
When you run the updater, WordPress displays the progress in an iframe. But when the PWA plugin is installed, I only see the offline template instead of the progress. That is very odd.
The text was updated successfully, but these errors were encountered: