midudev / la-velada-web-oficial

Web oficial de La Velada IV de Ibai Llanos
https://lavelada.es
Other
1.39k stars 609 forks source link

fix: don't request all images with workbox #860

Closed jamerrq closed 7 months ago

jamerrq commented 7 months ago

Descripción

Actualmente las imágenes se encuentran dentro del globPatterns del workbox (PWA)

Esto hace que en la primera petición que haces de la página (incluso si estás en incógnito o tienes la cache deshabilitada) se cargue absolutamente todas las imágenes del sitio lo cuál no creo que tenga mucho sentido para esta aplicación web en particular.

Más aún teniendo en cuenta que ya se registró una política de cacheo para las imágenes:

https://github.com/midudev/la-velada-web-oficial/blob/88e67821ee0bbe3b4727fb9671da0c611cca2ad5/astro.config.ts#L47-L57

Problema solucionado

Cambios propuestos

Capturas de pantalla (si corresponde)

image

Comprobación de cambios

[!NOTE] No he podido replicar los cambios localmente debido a una configuración de middleware, pero hice esto antes con mi sitio web personal y funcionó como esperaba

Impacto potencial

Reducir la carga inicial considerablemente

Contexto adicional

Enlaces útiles

vercel[bot] commented 7 months ago

@jamerrq is attempting to deploy a commit to the midudev pro Team on Vercel.

A member of the Team first needs to authorize it.