fix: manifest and favicons were returning 404s

favicon.ico and site.webmanifest should be in root of domain. Everything else goes into /static so it's easy to set TTL and other headers by path filter Fixes #424
amclin · Jun 26, 2020 · 92358de · 92358de
1 parent 21ebbcb
commit 92358de
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/templates/default/public/static/favicon.ico → templates/default/public/favicon.ico b/templates/default/public/static/favicon.ico → templates/default/public/favicon.ico
diff --git a/...es/default/public/static/site.webmanifest → templates/default/public/site.webmanifest b/...es/default/public/static/site.webmanifest → templates/default/public/site.webmanifest
@@ -3,12 +3,12 @@
     "short_name": "%%APPNAME%%",
     "icons": [
         {
-            "src": "/android-chrome-192x192.png",
+            "src": "/static/android-chrome-192x192.png",
             "sizes": "192x192",
             "type": "image/png"
         },
         {
-            "src": "/android-chrome-512x512.png",
+            "src": "/static/android-chrome-512x512.png",
             "sizes": "512x512",
             "type": "image/png"
         }

diff --git a/templates/default/src/components/molecules/Favicon/Favicon.jsx b/templates/default/src/components/molecules/Favicon/Favicon.jsx
@@ -8,19 +8,19 @@ const Favicon = () => {
         rel="apple-touch-icon"
         type="image/png"
         sizes="180x180"
-        href="/apple-touch-icon.png"
+        href="/static/apple-touch-icon.png"
       />
       <HeadLink
         rel="icon"
         type="image/png"
         sizes="32x32"
-        href="/favicon-32x32.png"
+        href="/static/favicon-32x32.png"
       />
       <HeadLink
         rel="icon"
         type="image/png"
         sizes="16x16"
-        href="/favicon-16x16.png"
+        href="/static/favicon-16x16.png"
       />
     </>
   )

diff --git a/templates/default/src/components/molecules/Favicon/__snapshots__/Favicon.test.jsx.snap b/templates/default/src/components/molecules/Favicon/__snapshots__/Favicon.test.jsx.snap
@@ -3,19 +3,19 @@
 exports[`Favicon creates a list of <link> elements for favicon suitable for the document head 1`] = `
 <DocumentFragment>
   <link
-    href="/apple-touch-icon.png"
+    href="/static/apple-touch-icon.png"
     rel="apple-touch-icon"
     sizes="180x180"
     type="image/png"
   />
   <link
-    href="/favicon-32x32.png"
+    href="/static/favicon-32x32.png"
     rel="icon"
     sizes="32x32"
     type="image/png"
   />
   <link
-    href="/favicon-16x16.png"
+    href="/static/favicon-16x16.png"
     rel="icon"
     sizes="16x16"
     type="image/png"