Skip to content

[FEATURE] Support du type de grain "transition" (PIX-17472) #12035

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

Merged
merged 2 commits into from
Apr 14, 2025

Conversation

yannbertrand
Copy link
Member

@yannbertrand yannbertrand commented Apr 11, 2025

🌸 Problème

En tant que contributeur de Modulix, je souhaite pouvoir créer un grain de type “transition” afin de dynamiser la lecture des modules.

🌳 Proposition

Mise en place et migration dans le bac-a-sable.

On propose que ces transitions n'aient pas avoir de titre obligatoire (contrairement aux autres types de grain). => Petit impact à prévoir, jusqu'à présent on remontait le "titre" du grain au dessus du texte de transition, proposition d'arrêter de faire ça.

Pour éviter de casser les tests e2e, ces titres vides sont cachés côté front en mode "moindre effort".

🐝 Remarques

Côté JSON Schema on est limités sur le "titre" obligatoire, j'ai du relacher un peu le schéma. On peut imaginer vérifier ces règles plus fines dans le modèle métier "Grain". Preneur des retours sur ce point.

L'extract CSV est géré.

Matomo est déjà géré puisqu'on enregistre déjà les clics sur "Continuer".

La navbar considère en l'état que les grains transitions sont des grains comme les autres donc on a "trop" d'étapes par rapport à la volonté future, et on se retrouve avec des Missing translation "pages.modulix.grain.tag.transition" for locale "fr". Je pense qu'on peut l'assumer tant que ça ne concerne que le bac-a-sable.

🤧 Pour tester

  • CI 🍏
  • S'assurer que les modules avec textes de transition classiques fonctionnent bien.
  • Vérifier que le bac-a-sable fonctionne en l'état (même si n'a pas l'attendu visuel final).

@yannbertrand yannbertrand self-assigned this Apr 11, 2025
@yannbertrand yannbertrand requested a review from a team as a code owner April 11, 2025 13:01
@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 via les liens suivants :

@yannbertrand yannbertrand force-pushed the PIX-17472-support-grain-transition branch from 81e0730 to ae81d69 Compare April 11, 2025 13:40
@dianeCdrPix dianeCdrPix force-pushed the PIX-17472-support-grain-transition branch from ae81d69 to f804e75 Compare April 14, 2025 08:50
@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-17472-support-grain-transition branch from f804e75 to 3a32029 Compare April 14, 2025 09:00
@pix-service-auto-merge pix-service-auto-merge merged commit 0d82cad into dev Apr 14, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-17472-support-grain-transition branch April 14, 2025 09:07
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.

6 participants