Merge branch 'sw-embed' into 'develop'

ServiceWorker: pass /embed through to the backend

See merge request soapbox-pub/soapbox-fe!1723

.eslintrc.js

@@ -78,7 +78,6 @@ module.exports = {
     'space-infix-ops': 'error',
     'space-in-parens': ['error', 'never'],
     'keyword-spacing': 'error',
-    'consistent-return': 'error',
     'dot-notation': 'error',
     eqeqeq: 'error',
     indent: ['error', 2, {
@@ -278,7 +277,6 @@ module.exports = {
       files: ['**/*.ts', '**/*.tsx'],
       rules: {
         'no-undef': 'off', // https://stackoverflow.com/a/69155899
-        'consistent-return': 'off',
       },
       parser: '@typescript-eslint/parser',
     },

webpack/production.js

@@ -92,7 +92,10 @@ module.exports = merge(sharedConfig, {
       cacheMaps: [{
         // NOTE: This function gets stringified by OfflinePlugin, so don't try
         // moving it anywhere else or making it depend on anything outside it!
-        match: ({ pathname }) => {
+        // https://github.com/NekR/offline-plugin/blob/master/docs/cache-maps.md
+        match: (url) => {
+          const { pathname } = url;
+
           const backendRoutes = [
             '/.well-known',
             '/activities',
@@ -119,10 +122,8 @@ module.exports = merge(sharedConfig, {
             '/unsubscribe',
           ];
 
-          if (pathname) {
+          if (backendRoutes.some(path => pathname.startsWith(path)) || pathname.endsWith('/embed')) {
-            return backendRoutes.some(path => pathname.startsWith(path));
+            return url;
-          } else {
-            return false;
           }
         },
         requestTypes: ['navigate'],