-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Make EIPs Website a PWA #5608
Comments
I'm concerned about caching because I don't understand PWAs. Is that something I should be worried about? |
Basically, you can implement bespoke caching rules. The one I'd implement is "stale while revalidate": Send the cached version immediately (if available) but update the cache in the background. |
I don't think this is an important use case and so it doesn't warrant more code that needs to be maintained. Unless I misunderstand the scope and this is only a few lines change. IMO a much bigger value add would just be improving the look of the EIPs website. |
It'll be about 30 lines of pretty understandable code (including comments). |
As much as I like PWA's loading speed improvement, not all search engines deal with PWA very well, so I am on the fence for adopting PWA for EIP site |
As implemented, it shouldn't impact search engine results. It will just be for browser caching. |
Some search engines don't index well PWA pages. E.g. here are a few things from one of the search engine. just to show it's nontrivial. https://developers.google.com/search/blog/2016/11/building-indexable-progressive-web-apps |
Pretty sure that that particular reference applies more to SPAs. This particular service worker is a glorified cache-control header and does not modify page content. |
Sure, if you understand the difference between PWA and SPA, just make sure the static HTTP yields the same or similar full content of text, then I have no concern at all for any PWA. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Still an issue. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
There's a PR open to add this. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Waiting on approval. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Waiting on approval. |
This is a comment to dismiss the stale review bot. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
This is a comment to dismiss the stale review bot. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
This is a comment to dismiss the stale review bot. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Would still like this. It doesn't add much complexity, and is really nice to have. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Dismissing bot. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Dismissing bot. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
Dismissing bot. |
There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity. |
I'll close this. It'll either happen or it won't. |
Proposed Change
This will allow for offline viewing of EIPs just by visiting the EIPs website. I volunteer to implement this.
The text was updated successfully, but these errors were encountered: