[BUGFIX] Inclure les images dans les builds, deuxième essai #668
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Une fois les applications déployées, elles seront accessibles via les liens suivants : Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-site-review-pr668/environment |
yannbertrand
force-pushed
the
fix-nuxt-image
branch
6 times, most recently
from
e3752ab to
21640da
Compare
yannbertrand
force-pushed
the
fix-nuxt-image
branch
from
5a1c936 to
21640da
Compare
VincentHardouin
approved these changes
Jun 18, 2024
yannbertrand
force-pushed
the
fix-nuxt-image
branch
from
5843e06 to
18cadb7
Compare
VincentHardouin
force-pushed
the
fix-nuxt-image
branch
2 times, most recently
from
b9e6596 to
fb5c76e
Compare
VincentHardouin
force-pushed
the
fix-nuxt-image
branch
from
4d62c59 to
06cceac
Compare
|
J'espère régénérer la RA pour éviter de mettre en cache le dossier |
|
Une fois les applications déployées, elles seront accessibles via les liens suivants : Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-site-review-pr668/environment |
|
Une fois les applications déployées, elles seront accessibles via les liens suivants : Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-site-review-pr668/environment |
yannbertrand
added
Tech Review OK
🚀 Ready to Merge
and removed
👀 Tech Review Needed
labels
This was referenced Jun 20, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🦄 Problème
Actuellement, les images ne sont plus incluses dans les build des différents sites. Cela a pour conséquence d'augmenter notre usage du CDN Prismic 💸 .
Suite à #661 on a eu des problèmes d'accès à certaines images...
Le 20/06/2024 en prod, sur la homepage, on constate 41 requêtes à des images n'étant pas sur
pix.fr(on requête divers CDN chez Prismic).🤖 Proposition
Utiliser @nuxt/image pour les images "dynamiques" (celles mises en place chez Prismic) pour qu'elles soient téléchargées lors du build.
Sur la Review App, il ne reste plus que 3 requêtes à des images hors
xx.review.pix.fr, car elles sont requêtées dynamiquement et pas à la compilation (voir le contenu deshared/components/NewsItemCard.vue). C'est déjà mieux.Point d'attention : 4 images sont en 404 sur la Review App.
Module+aventure-2.pngqui pareillement a été remplacé. (au passage on passe de ~15MB à ~1.5MB). Incertitude de pourquoi elle n'est pas présente non plus. A vérifier en intégration.🌈 Remarques
Nous avons finalement forker le projet pour s'adapter au mieux à l'encodage/ou pas fait par Prismic :
1024pix/nuxt-image@a7d030d
💯 Pour tester
ET/OU
Vérifier aussi que les images continuent de s'afficher.