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

feat(lunatic): format control for the year of date questions #1168

Merged
merged 6 commits into from
Dec 6, 2024

Conversation

nsenave
Copy link
Contributor

@nsenave nsenave commented Dec 5, 2024

Summary

cf. la user story :

À la saisie dans le composant Datepicker :

  • Lunatic "ajoute des 0" pour mettre la date au format : exemple 14 / 7 / 24 => la donnée est 0024-07-14
  • Lunatic bloque l'ajout de plus de 4 chiffres dans le champ de saisie

Le rôle du contrôle "de format" généré est de vérifier que l'année est saisie avec 4 chiffres (ici le répondant voulait sans doute dire "2024" et pas "l'an 24").

NB : rappel sur le vocabulaire métier des contrôles dans Bowie (Concevoir) :

  • Contrôle de cohérence = contrôle saisit par l'utilisateur dans Pogues (type CONSISTENCY in fine dans Lunatic)
  • Contrôle de format = contrôle saisit par l'utilisateur dans Pogues (type FORMAT in fine dans Lunatic)

Done

Ajout d'un contrôle de format VTL dans la génération Lunatic.

En VTL c'est un peu tricky (j'ai pas trouvé mieux) :

  1. convertir la donnée saisie que Lunatic collecte comme une string en date
  2. récupérer l'année de la date via une conversion en string avec le masque "YYYY"
  3. convertir en integer pour voir ça correspond bien à un nombre à 4 chiffres

NB : la partie vraiment utile du contrôle est celle qui vérifie qu'il y a au moins 4 chiffres, mais j'ai repris la même logique "ceinture et bretelles" que pour les InputNumber : même si le composant Lunatic bloque la saisie, le contrôle VTL vérifie le max.

@nsenave nsenave added the deploy-snapshot To be used in PR to trigger snapshot deploy pipeline label Dec 5, 2024
Copy link

github-actions bot commented Dec 5, 2024

👋 Version 3.30.0-SNAPSHOT deployed on docker hub

@github-actions github-actions bot removed the deploy-snapshot To be used in PR to trigger snapshot deploy pipeline label Dec 5, 2024
@nsenave nsenave added the deploy-snapshot To be used in PR to trigger snapshot deploy pipeline label Dec 5, 2024
Copy link

github-actions bot commented Dec 5, 2024

👋 Version 3.30.0-SNAPSHOT.1 deployed on docker hub

@github-actions github-actions bot removed the deploy-snapshot To be used in PR to trigger snapshot deploy pipeline label Dec 5, 2024
@nsenave nsenave merged commit 6cc98e1 into v3-next Dec 6, 2024
6 checks passed
@nsenave nsenave deleted the feat/improve-date-generated-control branch December 6, 2024 14:56
nsenave added a commit that referenced this pull request Dec 9, 2024
* feat(lunatic): format control for the year of date questions

* refactor: some java 21+ syntax refactor

* refactor: some clean up in test

* chore: bump version

* feat: add 'at least 4 digits' in control
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.

2 participants