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

✨ [POC] Erreur de validation dans les formulaires (cas des fichiers) #2241

Merged
merged 22 commits into from
Oct 2, 2024

Conversation

VioMrqs
Copy link
Collaborator

@VioMrqs VioMrqs commented Sep 23, 2024

Description

POC

  • Création d'un schéma de validation pour les documents
  • Application à tous les cas
  • Meilleure gestion et affichage des stateRelatedMessage en cas de problème de validation - ou pas (sous l'input directement)
  • Rapatriement des erreurs pour les dates dans les .action au lieu des .form
  • Typing propre 👍
  • Ajout de la petite librairie clsx (très légère)
  • Un peu de rangement 🧹

Ticket en lien

A faire

  • ajouter de meilleures check de validation pour les dates, les identifiants projets
  • Voir si possibilité de faire un quick-win dans le legacy

Type de changement

Veuillez supprimer les options qui ne sont pas pertinentes.

  • Correction de bug
  • Nouvelle fonctionnalité
  • Refacto de code
  • Breaking changes (correction ou fonctionnalité qui ferait en sorte que la fonctionnalité existante ne fonctionne pas comme prévu)
  • Ce changement nécessite une mise à jour de la documentation

Comment cela a-t-il été testé?

AVANT
Capture d’écran 2024-09-30 à 18 27 44

APRES

Capture d’écran 2024-09-30 à 18 26 04

@VioMrqs VioMrqs changed the title Fix/validation errors form ✨ Meilleure erreur de validation dans les formulaires - POC sur les fichiers Sep 23, 2024
@VioMrqs VioMrqs changed the title ✨ Meilleure erreur de validation dans les formulaires - POC sur les fichiers ✨ [POC] Erreur de validation dans les formulaires (cas des fichiers) Sep 23, 2024
@VioMrqs VioMrqs marked this pull request as ready for review September 23, 2024 13:19
@JulienPavon
Copy link
Collaborator

Est ce qu'il n'est pas possible d'avoir les erreurs au niveau des champs concernés comme on le fait pour les champs classiques ?

@VioMrqs
Copy link
Collaborator Author

VioMrqs commented Sep 24, 2024

On a des erreurs directement gérées par les input, ici on veut contrôler la taille ce qui n'est pas possible (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/file)

@benjlevesque
Copy link
Member

On a des erreurs directement gérées par les input, ici on veut contrôler la taille ce qui n'est pas possible (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/file)

je pense que ce que Julien propose, c'est d'afficher la réponse serveur en face du champs concerné. Ce n'est donc pas une validation client-side. Il suffit que ton retour serveur contienne le nom du champs concerné

@VioMrqs
Copy link
Collaborator Author

VioMrqs commented Sep 25, 2024

Ah yes je vois, je sais pas trop si je vais réussir à faire ça du coup, je regarde

@JulienPavon
Copy link
Collaborator

JulienPavon commented Sep 26, 2024

Ah yes je vois, je sais pas trop si je vais réussir à faire ça du coup, je regarde

En fait il suffit d'avoir le nom du champ et le message d'erreur associé dans le state du formulaire comme on fait déjà pour les erreur de validation zod

@VioMrqs VioMrqs force-pushed the fix/validation-errors-form branch 2 times, most recently from 487be5b to 139c7cc Compare September 27, 2024 09:18
@VioMrqs VioMrqs merged commit b5693d3 into main Oct 2, 2024
9 checks passed
@VioMrqs VioMrqs deleted the fix/validation-errors-form branch October 2, 2024 07:31
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.

3 participants