[FEATURE] Support du type de grain "transition" (PIX-17472) #12035
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🌸 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