Fix PWA manifest using `/web` paths (#19421)

lolsob-rspec
Eugen Rochko 2022-10-23 14:05:18 +02:00 committed by GitHub
parent daaa66a2c2
commit f1c016e7b6
2 changed files with 6 additions and 15 deletions

View File

@ -60,24 +60,15 @@ registerRoute(
self.addEventListener('install', function(event) {
event.waitUntil(Promise.all([openWebCache(), fetchRoot()]).then(([cache, root]) => cache.put('/', root)));
});
self.addEventListener('activate', function(event) {
event.waitUntil(self.clients.claim());
});
self.addEventListener('fetch', function(event) {
const url = new URL(event.request.url);
if (url.pathname.startsWith('/web/')) {
const asyncResponse = fetchRoot();
const asyncCache = openWebCache();
event.respondWith(asyncResponse.then(
response => {
const clonedResponse = response.clone();
asyncCache.then(cache => cache.put('/', clonedResponse)).catch();
return response;
},
() => asyncCache.then(cache => cache.match('/'))));
} else if (url.pathname === '/auth/sign_out') {
if (url.pathname === '/auth/sign_out') {
const asyncResponse = fetch(event.request);
const asyncCache = openWebCache();

View File

@ -52,7 +52,7 @@ class ManifestSerializer < ActiveModel::Serializer
end
def start_url
'/web/home'
'/home'
end
def scope
@ -77,11 +77,11 @@ class ManifestSerializer < ActiveModel::Serializer
[
{
name: 'Compose new post',
url: '/web/publish',
url: '/publish',
},
{
name: 'Notifications',
url: '/web/notifications',
url: '/notifications',
},
]
end