betagouv / mission-transition-ecologique

Application Transition Ecologique des Entreprises
https://mission-transition-ecologique.beta.gouv.fr/
GNU Affero General Public License v3.0
7 stars 7 forks source link

Ajouter des validateurs au niveau des champs du formulaire de contact #470

Closed ColineLebaratoux closed 5 months ago

ColineLebaratoux commented 10 months ago

Ajouter des validateurs pour éviter les erreurs 500 dans matomo :

ticket formulaire de contact lié : #470

jbledevehat commented 7 months ago

Un lien avec #627 ?

dolemoine commented 5 months ago

@dolemoine: implementer la validation du champs au focus out plutôt qu'à la soumission du formulaire

ColineLebaratoux commented 5 months ago

ticket lié : #262

dolemoine commented 5 months ago

@ColineLebaratoux est-ce que je traite le ticket #262 en même temps? auquel cas est-ce que les messages d'erreur et règles sont définis?

dolemoine commented 5 months ago

@ColineLebaratoux voilà le lien pour validaiton : https://tee-preprod-pr736.osc-fr1.scalingo.io/

ColineLebaratoux commented 5 months ago

Merci @dolemoine !

En testant, je me rends compte que c'est un peu lourd l'état de validation du DSFR. ça prend beaucoup de place ! Serait-il possible d'enlever la checkbox pour l'état valid ? et même le bandeau latéral si c'est possible ?

Image

dolemoine commented 5 months ago

@ColineLebaratoux c'est à jour sur l'url de demo: https://tee-preprod-pr736.osc-fr1.scalingo.io/

ColineLebaratoux commented 5 months ago

C'est parfait !

dolemoine commented 5 months ago

@ColineLebaratoux petite question sur les messges d'erreur suite à la review de Yohann:

Image

ColineLebaratoux commented 5 months ago

Zut, je n'ai plus ça en tête. Tu veux bien regénérer le lien de test ?

dolemoine commented 5 months ago

et voilà ! https://tee-preprod-pr736.osc-fr1.scalingo.io/

ColineLebaratoux commented 5 months ago

Capture d’écran 2024-05-27 à 12 57 05 ça semble pété sur ce lien, je n'arrive pas à accéder à une fiche dispositif 😕

dolemoine commented 5 months ago

@ColineLebaratoux c'est corrigé : https://tee-preprod-pr736.osc-fr1.scalingo.io/

jbledevehat commented 5 months ago

@ColineLebaratoux Ca te va si le champs est vide de mettre par défaut "Ce champs est obligatoire" plutôt que "...n'est pas valide" ?

ColineLebaratoux commented 5 months ago

Si j'essaye d'interpréter le commentaire de @yvalentin, il s'agirait d'afficher "ce champs est obligatoire" lorsque rien n'a été saisi dans le champ. et de mettre "...n'est pas valide" lorsque le format n'est pas le bon. C'est ok pour moi. Tu confirmes Yoh ?

dolemoine commented 5 months ago

je crois que c'est l'idée oui. Ok pour moi, ça demande quelques petites modifications, je regarde ça demain!

yvalentin commented 5 months ago

Si j'essaye d'interpréter le commentaire de @yvalentin, il s'agirait d'afficher "ce champs est obligatoire" lorsque rien n'a été saisi dans le champ. et de mettre "...n'est pas valide" lorsque le format n'est pas le bon. C'est ok pour moi. Tu confirmes Yoh ?

Oui, c'est bien ça ;-)

dolemoine commented 5 months ago

@ColineLebaratoux dernière version avec la petite modif dispo sur le lien de test : https://tee-preprod-pr736.osc-fr1.scalingo.io/