Validation des numéros ANTS des proches

[adipasquale]

Closes #4777

Contexte

Actuellement les numéros de pré-demande ANTS des proches ne sont pas du tout validés.

Cette PR fait suite à ces PR de préparation : #4770 et #4782 qui ont permis d’extraire la logique de validation des numéros de pré–demande ANTS.

Solution

Dans cette PR on rajoute de la validation sur les champs de numéros de pré-demande ANTS des proches.

Le gros changement est qu’on introduit un nouveau form object RelativeUserForm.
Il délègue plusieurs attributs au user passé en param, dont le ants_pre_demande_number.
Le form object rajoute la validation du numéro de pré-demande quand c’est demandé.

Captures d’écran

Review app

RDV sur cette url :

https://demo-rdv-solidarites-pr4824.osc-secnum-fr1.scalingo.io/prendre_rdv/?departement=95&city_code=95582&street_ban_id=&latitude=48.973523&longitude=2.253694&address=Sannois%2C+95110&button=

et vous pouvez vous connecter avec [email protected] : Rdvservicepublictest1!

et utiliser les numéros ANTS RDVSPUB001 à 004

PRs suivantes

• modification du message pour préciser que le numéro est obligatoire et pas recommandé
• amélioration de l’UI pour afficher le numéro de pré-demande ANTS des proches dans le flow de prise de RDV et permettre de l’éditer
• migration pour corriger les numéros de demande existants dans la DB
• amélioration du flow ANTS pour que les durées de créneaux correspondent au nombre de participants

[adipasquale]

je fais le parsing du booléen ici même si ce n’est pas super conventionnel

[adipasquale]

ça permet de propager les erreurs au formulaire pour qu’elles s’affichent correctement

[francois-ferrandis]

Ça m'a l'air très bien ! 🚀

Ça me donne envie de lire des guides pour mieux comprendre les enjeux derrière les form objects. J'ai l'impression que dans le cas présent c'est utile pour :

• activer ou non la validation via ants_pre_demande_number_required, qui n'est pas un attribut de l'usager mais bin un élément de contexte
• utiliser validates_with AntsPreDemandeNumberValidation dans ce contexte mais pas directement sur le modèle User (de manière générale je crois que les form objects permettent d'extraire les validations du modèle)

[adipasquale]

merci @francois-ferrandis 🙇

En effet ici l’intérêt du form object est de :

• signifier clairement que ce n’est pas un formulaire de user « complet » mais formulaire pour un certain type de user : les proches. Ces formulaires ne présentent pas autant de champs que le formulaire classique
• pouvoir rajouter de la validation contextuellement. À ce sujet, j’ai hésité à faire deux form objects : RelativeUserForm et RelativeUserWithAntsPreDemandeNumberForm, mais j’ai choisi d’en faire un seul avec un attribut transitoire, c’est discutable