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] Permettre de créer de nouvelles pages avec des slices sur Prismic (PIX-1169) #156

Merged
merged 4 commits into from
Sep 7, 2020

Conversation

celineung
Copy link
Contributor

@celineung celineung commented Sep 2, 2020

🦄 Problème

L'équipe de communication doit pouvoir, via Prismic, créer de nouvelles pages qui utilisent les nouveaux slices existants (Banner, Feature et Article).

🤖 Solution

  • Création d'un nouveau Custom Type Prismic: slices_page
  • côté app: création d'une nouvelle page _custom-page.vue, qui, en fonction du type de Custom Type (simple_page ou slices_page), appelera le composant SimplePage ou SliceZone

🌈 Remarques

  • le template de SimplePage a été déplacé dans components. Est-il nécessaire de le renommer?
  • deux champs header et footer figuraient sur le Custom Type simple_page de Prismic mais n'étaient pas prise en compte dans l'app. Par conséquent, ils ont été supprimés.

✨ Review App

@celineung celineung self-assigned this Sep 2, 2020
@celineung celineung force-pushed the PIX-1169-dynamic-slices-in-page branch 2 times, most recently from 3e08696 to dc01a25 Compare September 2, 2020 09:16
@celineung celineung marked this pull request as ready for review September 2, 2020 09:21
tests/services/document-fetcher.test.js Outdated Show resolved Hide resolved
services/document-fetcher.js Show resolved Hide resolved
pages/_custom-page.vue Show resolved Hide resolved
@celineung celineung force-pushed the PIX-1169-dynamic-slices-in-page branch from dc01a25 to f2544e0 Compare September 2, 2020 16:40
@celineung celineung force-pushed the PIX-1169-dynamic-slices-in-page branch 2 times, most recently from f4f776f to 1e5a03c Compare September 4, 2020 08:37
pages/_custom-page.vue Outdated Show resolved Hide resolved
pages/_custom-page.vue Outdated Show resolved Hide resolved
pages/_custom-page.vue Outdated Show resolved Hide resolved
tests/services/document-fetcher.test.js Outdated Show resolved Hide resolved
@celineung celineung force-pushed the PIX-1169-dynamic-slices-in-page branch 6 times, most recently from 1221e45 to 7ab74b2 Compare September 4, 2020 13:39
@celineung celineung force-pushed the PIX-1169-dynamic-slices-in-page branch from 7ab74b2 to 59df8e9 Compare September 7, 2020 08:11
pages/_custom-page.vue Show resolved Hide resolved
pages/_custom-page.vue Show resolved Hide resolved
@sbedeau
Copy link
Contributor

sbedeau commented Sep 7, 2020

Ok techniquement mais je n'ai pas réussi à tester fonctionnellement

@celineung celineung force-pushed the PIX-1169-dynamic-slices-in-page branch from 59df8e9 to 810bb76 Compare September 7, 2020 12:52
@celineung celineung merged commit 9821a09 into dev Sep 7, 2020
@celineung celineung deleted the PIX-1169-dynamic-slices-in-page branch September 7, 2020 15: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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants