search

AtB-AS / webshop

AtB Webshop / AtB Nettbutikk
https://nettbutikk.atb.no
European Union Public License 1.2
0 stars 1 forks source link

chore: remove workbox and sw generation #481

Closed mikaelbr closed 2 years ago

mikaelbr commented 2 years ago

Nå er det lagt inn en autogenerering av en Service Worker som traverserer alle static assets og prefetcher de. Fordelen er at dersom du kommer over et bilde så vil det ikke være et midlertidig øyeblikk hvor bildet lastes (i de fleste tilfeller).

Ulempen her er at vi har ingen god måte å vite hvilke bilder som er i bruk eller ikke (uten å manuelt vedlikeholde det). Tradisjonelt sett kan dette gjøres med å bruke webpack og loading av filer via deres modulsystem som gjør at du vet hvilke filer som skal flyttes over. Her via Elm så er det ikke noe støtte for importing og generering av relative paths for bruk av bilder til img-tags. Dvs at vi bare refererer løst til filer som ligger i en mappe.

Tidligere betydde det noen ekstra bilder som ble lastet inn som brukeren egentlig ikke hadde behov for. Nå med flere ikoner flyttet over til design systemet (og på sikt sikkert flere ikoner som flyttes dit) så betyr det mange ikoner som finnes i static mappen som teknisk sett ikke brukes. Det fører til at sluttbruker laster mange filer de ikke har behov for (se https://github.com/AtB-AS/webshop/issues/470#issuecomment-1035958718).

Forslag til å fjerne SW-generering i det hele tatt. Vi kunne ha inkludert JS og CSS filer her, men det har ingen hensikt ettersom det er CSR og ingen tree shaking som gjør at når du åpner siden vil du automatisk cache komplett js og css uansett.

github-actions[bot] commented 2 years ago

Visit the preview URL for this PR (updated for commit 7928a33):

https://atb-webshop-staging--pr481-mikael-remove-workbo-rg19os9x.web.app

(expires Sun, 13 Feb 2022 08:55:57 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

gorandalum commented 2 years ago

Jeg tenker dette går fint. Brukerne besøker nettsiden for et formål og derav er neppe lastetiden på bilder like essensiell for om de fortsetter sesjonen som det kan være på andre nettsider.