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

[FEATURE] Renvoyer vers les bonnes locales quand les pages de prismic référencent des liens vers d'autres pages (PIX-4903). #393

Merged
merged 5 commits into from
May 6, 2022

Conversation

VincentHardouin
Copy link
Member

@VincentHardouin VincentHardouin commented May 5, 2022

🦄 Problème

Actuellement, si une slices pages est référencée dans une page Prismic, la redirection ne fonctionnera pas correctement. Il en est de même avec les actualités (type: NEWS_ITEM) où on ne rajoute que le préfix actualites mais pas forcément news pour la locale en-gb par exemple.

🤖 Solution

  • Ajouter des tests sur le link-resolver
  • Gérer les locales dans le link-resolver
  • Gérer les préfix pour les pages NEWS_ITEM

🌈 Remarques

Des infos supplémentaires, trucs et astuces ?

💯 Pour tester

  • Créer une page qui a un lien vers un autre document
  • Vérifier que le lien renvoie bien vers la locale

@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

@VincentHardouin VincentHardouin self-assigned this May 5, 2022
@VincentHardouin VincentHardouin added 👀 Tech Review Needed team-evaluation PR relatives à l'expérience d'évaluation labels May 5, 2022
@alexandrecoin
Copy link
Contributor

Tech OK 👍

@VincentHardouin VincentHardouin force-pushed the pix-4903-improve-link-resolver branch from e5cb819 to c7e0507 Compare May 5, 2022 14:51
@VincentHardouin VincentHardouin force-pushed the pix-4903-improve-link-resolver branch from 1dddc44 to d0cd6e4 Compare May 6, 2022 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 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.

5 participants