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

Quote validator v0 avec des boucles if sur la présence des infos #17

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

martin-letellier
Copy link

No description provided.

@martin-letellier
Copy link
Author

@nicolasleger voici une V0 pour la vérification de présence des élements.

  • Une fonction pour vérifier les info administrative
  • Une fonction pour vérifier tous les gestes avec un gros case when pour faire les vérifs en fonction du geste.

TODO : Vérifier les valeurs / Voir si on doit rajouter un peu plus d'infos dans le errors pour pouvoir créer le wording ensuite et avoir le contexte du geste par exemple.

Dis-moi ce que tu en penses.
Merci

Copy link
Collaborator

@nicolasleger nicolasleger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top, on pourrait faire des concerns ou même des sous-classe par partie Isolation...

@martin-letellier
Copy link
Author

@nicolasleger c'est good, j'ai tout séparé en sous-classe.

Le validator global s'occupe de la partie admin et de la boucle each sur tous les travaux.
Ensuite, il appelle chaque sous classe en fonction du geste identifié et empile les erreurs.

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.

2 participants