-
Notifications
You must be signed in to change notification settings - Fork 52
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
Cache just the home page. Fix caching of offline page #852
Conversation
…n the page. Create a new cache for caching just the homepage. Clear the old cache routes to reduce cache size.
@@ -0,0 +1,17 @@ | |||
{% extends "layout.html.jinja2" %} | |||
{% block title %}{% trans %}Logout{% endtrans %}{% endblock %} | |||
|
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.
This needs a bit of content explaining that the logout is in progress. We have a similar case in meta_refresh.html.jinja2
.
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.
This helps, but I was referring to the body. Users will see a blank page for a second or so. We might as well show "Logging out…" in the middle of the screen for that second.
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.
This needs a logout message in the body of the page (not just title). Otherwise all okay.
* Cache just the home page. Fix caching of offline page * On logout, render a logout template and clear service worker caches in the page. Create a new cache for caching just the homepage. Clear the old cache routes to reduce cache size. * Don't cache the JS files. Delete the cache routes and precache * Move clearing cache to homepage * Change logout page title * Remove flash message
No description provided.