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] Récupérer les meta descriptions depuis prismic. #240

Merged
merged 3 commits into from
Feb 19, 2021

Conversation

jonasgrilleres
Copy link
Contributor

@jonasgrilleres jonasgrilleres commented Feb 17, 2021

🦄 Problème

Les meta descriptions proviennent du package.json et il n'est pas possible de les changer pour chaque page.

🤖 Solution

On récupère la meta description depuis prismic. Si elle n'est pas renseignée, on utilise une description générique.

💯 Pour tester

  • Lancer en local avec la commande SSR_ENABLED=true npm run dev:site
  • Lancer la preview depuis prismic

Remplir une description dans l'onglet Social Cards et voir qu'elle s'applique bien au tag <meta name="description" content="Ma description">

Si rien n'est rempli, on doit y voir la phrase par défaut.

@pix-service
Copy link

I'm deploying this PR to these urls:

Please check it out!

@jonasgrilleres jonasgrilleres force-pushed the fetch-meta-description-from-prismic branch 6 times, most recently from feeea29 to e677c0a Compare February 17, 2021 14:19
@jonasgrilleres jonasgrilleres changed the title Fetch meta description from prismic [FEATURE] Récupérer les meta descriptions depuis prismic. Feb 17, 2021
@jonasgrilleres jonasgrilleres force-pushed the fetch-meta-description-from-prismic branch from e677c0a to aeeb00d Compare February 17, 2021 14:28
@jonasgrilleres jonasgrilleres marked this pull request as ready for review February 17, 2021 14:28
@aceol
Copy link
Contributor

aceol commented Feb 17, 2021

Cette methode necessite de definir des meta pour chaque page. On pourrais avoir un fallback pour pix-site et pix-org par langue

@jonasgrilleres jonasgrilleres force-pushed the fetch-meta-description-from-prismic branch from aeeb00d to b11e36e Compare February 19, 2021 09:12
@jonasgrilleres jonasgrilleres merged commit 258b048 into dev Feb 19, 2021
@jonasgrilleres jonasgrilleres deleted the fetch-meta-description-from-prismic branch February 19, 2021 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants