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

Add no_url and no_phone_number validations on Campaign#extra_info #790

Conversation

alexandreruban
Copy link
Collaborator

@alexandreruban alexandreruban commented May 16, 2021

Fixes #773

Résumé

On ajoute deux validations au champ informations supplémentaires des campagnes (extra_info) pour éviter qu'il contienne une URL ou un numéro de téléphone pour prendre rdv.

Détails

⚠️ Warning : ne mettre en prod que lorsque le numéro de téléphone des lieux de vaccination seront - s'ils le souhaitent - disponible dans la page de confirmation d'un match, afin que des volontaires puissent tout de même les contacter et que nous ne bloquions pas tous les centres qui souhaiteraient laisser leur coordonnées de contact.

J'ai ajouté les validations demandées, mais par contre le message d'erreur n'est pas clair du tout pour le moment.

Screenshot 2021-05-16 at 22 54 15

@alexandreruban alexandreruban marked this pull request as draft May 16, 2021 20:57
@mininao
Copy link
Member

mininao commented May 16, 2021

Toute la validation de ce form se fait côté client, pour moi le check est à faire côté React. Je peux le reprendre si tu es pas à l'aise avec 🙂

@arnaudriegert
Copy link
Contributor

⚠️ Warning : ne mettre en prod que lorsque le numéro de téléphone des lieux de vaccination seront - s'ils le souhaitent - disponible dans la page de confirmation d'un match, afin que des volontaires puissent tout de même les contacter et que nous ne bloquions pas tous les centres qui souhaiteraient laisser leur coordonnées de contact.

--> #793 🙂

@alexandreruban
Copy link
Collaborator Author

@mininao oui effectivement !

Je m'attendais tellement à avoir un formulaire classique en Rails que je suis parti sur le back direct ! Si jamais tu veux reprendre, pas de souci, je ne connais mal les libs React pour valider par exemple des numéros de tel alors tu t'en sortiras sans doute mieux que moi ! 😄

Si jamais cette PR ne sert à rien, je te laisse la clôturer et je te laisse te réassigner sur l'issue ? C'est bon pour toi ?

@mininao
Copy link
Member

mininao commented May 17, 2021

Je fais ça, ça marche ! Merci @alexandreruban !

@mininao mininao closed this May 17, 2021
@mininao
Copy link
Member

mininao commented May 17, 2021

(J'ai laissé l'issue sans owner, je verrais si je prends ou si quelqu'un d'autre est chaud avant 🙂)

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.

Ajouter une vérification du champ d'information pour éviter les demandes de RDV de la part des centres
3 participants