Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUGFIX] Réparer l'affichage des images en preview (PIX-14766) #701

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

Jeyffrey
Copy link
Contributor

@Jeyffrey Jeyffrey commented Aug 30, 2024

🦄 Problème

L'affichage de nouvelles images ne s'affichent plus lors des preview Prismic.

Cela est causée par les changements effectuées pour intégrer les images au build de l'application.
Les nouvelles images en preview ne sont pas intégrées au build ce qui cause ce problème.

🤖 Proposition

Ajouter une variable d'environnement IS_BUILTIN_IMAGES_ENABLED pour gérer ce comportement.

  • Dans le cas où la variable est à false, alors la fonctionnalité d'intégration des images au build de l'application est désactivée pour permettre à la preview de fonctionner correctement.

🌈 Remarques

RAS

💯 Pour tester

  • Sur Prismic, ajouter une nouvelle image à la banque d'images.
  • Modifier un document avec le tag pix-site en ajoutant cette nouvelle image.
  • Tester la preview avec le bouton Preview the page en choisissant Test images (pix.fr)
  • Vérifier que l'image s'affiche bien sur la preview.
  • Modifier un document avec le tag pix-pro en ajoutant cette nouvelle image.
  • Tester la preview avec le bouton Preview the page en choisissant Test images (pro.fr)
  • Vérifier que l'image s'affiche bien sur la preview.

@pix-bot-github
Copy link

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-pr701/environment

@Jeyffrey Jeyffrey force-pushed the fix-nuxt-image-provider-2 branch 3 times, most recently from 8b3455e to 5083dcb Compare August 30, 2024 12:23
shared/nuxt.config.ts Outdated Show resolved Hide resolved
@Jeyffrey Jeyffrey changed the title fix(NuxtImage): use prismic provider on preview [BUGFIX] Réparer l'affichage des images en preview (PIX-14766) Oct 10, 2024
@er-lim er-lim force-pushed the fix-nuxt-image-provider-2 branch 2 times, most recently from 4fd9105 to 96fd1b3 Compare October 10, 2024 13:45
@er-lim er-lim force-pushed the fix-nuxt-image-provider-2 branch from 96fd1b3 to c62154c Compare October 10, 2024 13:56
@Jeyffrey Jeyffrey marked this pull request as ready for review October 10, 2024 14:21
@Jeyffrey Jeyffrey added Tech Review OK team-evaluation PR relatives à l'expérience d'évaluation cross-team Toutes les équipes de dev and removed 🚧 Development in progress labels Oct 10, 2024
@pix-service-auto-merge pix-service-auto-merge merged commit 0312160 into dev Oct 10, 2024
10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the fix-nuxt-image-provider-2 branch October 10, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 🚀 Ready to Merge team-evaluation PR relatives à l'expérience d'évaluation Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants