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] Corriger la preview Prismic sur les fragments de documents (PIX-11601) #637

Merged
merged 1 commit into from
Apr 19, 2024

Conversation

er-lim
Copy link
Contributor

@er-lim er-lim commented Mar 12, 2024

🦄 Problème

Actuellement la preview Prismic ne fonctionne pas sur les documents qui sont des fragments (ex: footer, nav...).
Elle retourne un undefined dans l'url

🤖 Proposition

Faire fonctionner la preview sur ces documents en corrigeant ce comportement.

🌈 Remarques

RAS

💯 Pour tester

  • Se connecter à Prismic
  • Se mettre sur la locale French France
  • Aller sur les détails du document nav Pix V2
  • Cliquer sur le bouton Preview the page en bas à gauche de la page

Reproduire le bug

  • Sélectionner Integration dans la liste des preview
  • Constater que l'url généré a un path en /undefined et que la page ne s'affiche pas correctement

Fix

  • Sélectionner PR 637 fr dans la liste des preview
  • Constater que l'url généré a un path en / et que la home s'affiche bien.

@er-lim er-lim added 🚧 Development in progress cross-team Toutes les équipes de dev labels Mar 12, 2024
@er-lim er-lim self-assigned this Mar 12, 2024
@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-pr637/environment

@er-lim er-lim changed the title fix(site): correct link resolver returning undefined [BUGFIX] Corriger la preview Prismic sur les fragments de documents (PIX-11601) Mar 12, 2024
@er-lim er-lim force-pushed the pix-11601-fix-prismic-fragment-preview branch from 653eedf to 93605c9 Compare March 26, 2024 13:18
@er-lim er-lim marked this pull request as ready for review April 2, 2024 07:58
Copy link
Contributor

@igorissen igorissen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test ✅

@er-lim er-lim force-pushed the pix-11601-fix-prismic-fragment-preview branch from 93605c9 to 61e3817 Compare April 19, 2024 08:10
@pix-service-auto-merge pix-service-auto-merge merged commit 083a9ed into dev Apr 19, 2024
5 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-11601-fix-prismic-fragment-preview branch April 19, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants