Skip to content
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

Closed
Pandapip1 opened this issue Sep 6, 2022 · 37 comments
Closed

Make EIPs Website a PWA #5608

Pandapip1 opened this issue Sep 6, 2022 · 37 comments
Labels
enhancement r-website Relates to the Website w-stale Waiting on activity

Comments

@Pandapip1
Copy link
Member

Proposed Change

This will allow for offline viewing of EIPs just by visiting the EIPs website. I volunteer to implement this.

@SamWilsn
Copy link
Contributor

SamWilsn commented Sep 7, 2022

I'm concerned about caching because I don't understand PWAs. Is that something I should be worried about?

@Pandapip1
Copy link
Member Author

Pandapip1 commented Sep 7, 2022

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.

@lightclient
Copy link
Member

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.

@Pandapip1
Copy link
Member Author

Unless I misunderstand the scope and this is only a few lines change.

It'll be about 30 lines of pretty understandable code (including comments).

@xinbenlv
Copy link
Contributor

xinbenlv commented Sep 7, 2022

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

@Pandapip1
Copy link
Member Author

not all search engines deal with PWA very well

As implemented, it shouldn't impact search engine results. It will just be for browser caching.

@xinbenlv
Copy link
Contributor

xinbenlv commented Sep 8, 2022

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

@Pandapip1
Copy link
Member Author

Pandapip1 commented Sep 8, 2022

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.

@xinbenlv
Copy link
Contributor

xinbenlv commented Sep 8, 2022

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.

@Pandapip1 Pandapip1 linked a pull request Sep 8, 2022 that will close this issue
@Pandapip1 Pandapip1 changed the title Make EIPs Website a PWA Proposal: Make EIPs Website a PWA Sep 10, 2022
@Pandapip1 Pandapip1 changed the title Proposal: Make EIPs Website a PWA Make EIPs Website a PWA Sep 10, 2022
@Pandapip1 Pandapip1 added the r-website Relates to the Website label Sep 10, 2022
@github-actions
Copy link

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Sep 22, 2022
@Pandapip1
Copy link
Member Author

Still an issue.

@github-actions github-actions bot removed the w-stale Waiting on activity label Sep 26, 2022
@github-actions
Copy link

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Nov 23, 2022
@Pandapip1
Copy link
Member Author

There's a PR open to add this.

@github-actions github-actions bot removed the w-stale Waiting on activity label Nov 24, 2022
@github-actions
Copy link

github-actions bot commented Dec 1, 2022

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Dec 1, 2022
@Pandapip1
Copy link
Member Author

Waiting on approval.

@github-actions github-actions bot removed the w-stale Waiting on activity label Dec 3, 2022
@github-actions
Copy link

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Dec 10, 2022
@Pandapip1
Copy link
Member Author

Waiting on approval.

@Pandapip1
Copy link
Member Author

This is a comment to dismiss the stale review bot.

@github-actions github-actions bot removed the w-stale Waiting on activity label Jan 14, 2023
@github-actions
Copy link

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Jan 21, 2023
@Pandapip1
Copy link
Member Author

This is a comment to dismiss the stale review bot.

@github-actions github-actions bot removed the w-stale Waiting on activity label Jan 22, 2023
@github-actions
Copy link

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Jan 31, 2023
@Pandapip1
Copy link
Member Author

This is a comment to dismiss the stale review bot.

@github-actions github-actions bot removed the w-stale Waiting on activity label Feb 2, 2023
@github-actions
Copy link

github-actions bot commented Feb 9, 2023

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Feb 9, 2023
@Pandapip1
Copy link
Member Author

Would still like this. It doesn't add much complexity, and is really nice to have.

@github-actions github-actions bot removed the w-stale Waiting on activity label Feb 10, 2023
@github-actions
Copy link

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Feb 17, 2023
@Pandapip1
Copy link
Member Author

Dismissing bot.

@github-actions github-actions bot removed the w-stale Waiting on activity label Feb 18, 2023
@github-actions
Copy link

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Feb 26, 2023
@Pandapip1
Copy link
Member Author

Dismissing bot.

@github-actions github-actions bot removed the w-stale Waiting on activity label Feb 27, 2023
@github-actions
Copy link

github-actions bot commented Mar 6, 2023

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Mar 6, 2023
@Pandapip1
Copy link
Member Author

Dismissing bot.

@github-actions github-actions bot removed the w-stale Waiting on activity label Mar 7, 2023
@github-actions
Copy link

There has been no activity on this issue for 1 week. It will be closed after 3 months of inactivity.

@github-actions github-actions bot added the w-stale Waiting on activity label Mar 15, 2023
@Pandapip1
Copy link
Member Author

I'll close this. It'll either happen or it won't.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement r-website Relates to the Website w-stale Waiting on activity
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants
@xinbenlv @lightclient @Pandapip1 @SamWilsn and others