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

Aucune validation sur les numéros de pré-demande ANTS des proches #4777

Open
adipasquale opened this issue Nov 5, 2024 · 0 comments
Open
Assignees
Labels
ANTS interconnexion ANTS mairie recherche de créneaux et dédoublonnage

Comments

@adipasquale
Copy link
Contributor

❓ Définition du problème

Lorsqu’un usager créé un proche pendant la prise de RDV, il doit ajouter un numéro de pré-demande mais aucune validation n’est effectuée (ni validation de format ni vérifications de dédoublonnage).

Cela cause des erreurs string does not match regex sur Sentry , par exemple https://sentry.incubateur.net/organizations/betagouv/issues/128894

💡 Solution envisagée

Je propose cette direction pour corriger ce souci :

  1. déplacer les validations de format au niveau du modèle User
  2. rajouter des vérifications de dédoublonnage lors de la création de RDV avec des proches avec des numéros

1. déplacer les validations de format au niveau du modèle User

Je ne vois pas de bonne raison de permettre de stocker des numéros de pré-demande aux formats invalides dans notre DB.
En rajoutant la validation au niveau du modèle on devrait se prémunir durablement de ces soucis.
Il faut suivre plusieurs étapes et déploiements pour gérer l’existant en douceur :

  1. rajouter la validation mais ne l’appliquer que sur les créations et modifications pour ne pas gêner les mises à jour de fiches usagers existantes avec des numéros mal formattés
  2. faire une migration pour supprimer tous les numéros de pré-demande erronés
  3. rendre la validation ubiquitaire

2. rajouter des vérifications de dédoublonnage lors de la création de RDV avec des proches avec des numéros

Je pense qu’une bonne piste est d’utiliser un form object pour pouvoir rajouter des validations uniquement à certains endroits de l’application.
On utilise déjà ce pattern dans l’application à divers endroits.

Cela implique plusieurs changements :

  1. génériser la validation existante en un vrai validateur
  2. réparer le form model de la step1 qui est utilisé de manière contournée - on affiche un formulaire basé sur le user plutôt que sur le form model
  3. créer et utiliser un form model pour la page de création des proches (en modale)
  4. rajouter la validation à ce form model

cf #4770 qui commence ce travail

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ANTS interconnexion ANTS mairie recherche de créneaux et dédoublonnage
Projects
Status: 🔖 Ready
Development

No branches or pull requests

1 participant