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

PWA more offline availability #1000

Merged
merged 12 commits into from
Jul 16, 2023
Merged

PWA more offline availability #1000

merged 12 commits into from
Jul 16, 2023

Conversation

felixrindt
Copy link
Member

@felixrindt felixrindt commented Jul 14, 2023

This updates the PWA views to class based views and adds a few more fields. Then there are some random fixes and touchups. Biggest change (and the most debatable) is the new serviceworker that caches sites you visit on the go.

@felixrindt felixrindt added [C] enhancement Changes to an existing feature making it better [P] nice to have features that are not so important labels Jul 14, 2023
@coveralls
Copy link

coveralls commented Jul 14, 2023

Coverage Status

coverage: 88.457% (+0.1%) from 88.351% when pulling 5bd27af on pwa-improvements into 503db27 on main.

@felixrindt felixrindt marked this pull request as ready for review July 14, 2023 17:58
@jeriox
Copy link
Contributor

jeriox commented Jul 16, 2023

When trying this with the offline simulator in Chrome Desktop, the CSS of the offline page is not rendered. Service worker has been freshly installed.

Also on the cached pages there is no offline banner visible
grafik
grafik

@felixrindt felixrindt enabled auto-merge (squash) July 16, 2023 19:57
@felixrindt felixrindt merged commit b2b2ace into main Jul 16, 2023
16 checks passed
@felixrindt felixrindt deleted the pwa-improvements branch July 16, 2023 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] enhancement Changes to an existing feature making it better [P] nice to have features that are not so important
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants