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] Pages en Erreur 404 sur les sites .fr et .org (PIX-1611) #216

Merged
merged 1 commit into from
Nov 16, 2020

Conversation

bpetetot
Copy link
Contributor

🦄 Problème

Certaines pages en ligne sur Prismic qui sont en Erreur 404 sur les sites .fr et .org : les pages Protection des données à caractère personnel et la Liste des sous-traitants.

Elles ne sont pas référencées dans la nav ou le footer, mais par un lien dans les CGU vers la Politique de protection, et un lien dans la Politique de protection vers la Liste des sous-traitants.

🤖 Solution

Ces 2 liens (Protection des données à caractère personnel et la Liste des sous-traitants) sont actuellement des liens de type "Web" vers pix.fr dans des éléments RichText de Prismic. Par conséquence, à la génération statique, ces liens ne sont pas parcouru et donc nuxt ne génère pas ces pages.

Nous avons déjà un mécanisme mis en place afin de transformer les liens de type Document des RichText Prismic. Mais ce mécanisme a un bug pour les liens de type document car ils sont transformés en nuxt-link qui ne sont pas interprété.

Pour corriger ce bug, nous avons mis en place cette solution (utilisé dans le projet d'exemple nuxt de prismic):
nuxt-modules/prismic#60

Les liens sont transformés en lien marqué nuxt-link (<a data-nuxt-link>) puis ces liens sont gérés par le plugin prismicLink.

🌈 Remarques

Les liens "Protection des données à caractère personnel" et la "Liste des sous-traitants" ont déjà été changé de type Web en type Document.

💯 Pour tester

  1. Aller sur la page des CGU (lien dans le footer)
  2. Cliquer sur le lien "Politique de protection des données à caractère personnel" (Chapitre 12)
  3. Cliquer sur le lien "Lien vers la liste des sous-traitants" (Chapitre "Sous-traitants du GIP Pix")

@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

Copy link
Contributor

@HEYGUL HEYGUL left a comment

Choose a reason for hiding this comment

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

Top !
Testé fonctionnellement également 👍

@bpetetot bpetetot force-pushed the fix-links-in-rich-texts branch from 10d2499 to 2d3aac4 Compare November 16, 2020 14:59
@bpetetot bpetetot merged commit 4ac0fc2 into dev Nov 16, 2020
@bpetetot bpetetot deleted the fix-links-in-rich-texts branch November 16, 2020 15:18
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