You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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).
Je propose cette direction pour corriger ce souci :
déplacer les validations de format au niveau du modèle User
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 :
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
faire une migration pour supprimer tous les numéros de pré-demande erronés
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 :
génériser la validation existante en un vrai validateur
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
créer et utiliser un form model pour la page de création des proches (en modale)
❓ 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
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 :
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 :
cf #4770 qui commence ce travail
The text was updated successfully, but these errors were encountered: