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

Adding verification on required fields #50

Closed
wants to merge 2 commits into from
Closed

Adding verification on required fields #50

wants to merge 2 commits into from

Conversation

emnoze
Copy link

@emnoze emnoze commented Apr 10, 2020

Permet de ne pas pouvoir valider le formulaire si jamais tout les champs ne sont pas remplis.

@Sicault

This comment has been minimized.

@NerOcrO
Copy link

NerOcrO commented Apr 11, 2020

Pourquoi ne pas avoir utiliser l'attribut HTML required ?

@strokyl
Copy link

strokyl commented Apr 11, 2020

Mince, j'ai fait la même PR en //:
#69
Sinon @NerOcrO j'ai pensé à ça mais il aura fallu traduire tout leur "validateur" fait en la main en JS en validateur HTML5. J'ai préféré pas trop changer de chose.
Par contre dans ma PR, plutôt que de faire une alert, je me contente de désactivé le boutton (ça couleur devient bleu claire).
Je vérifie aussi que les champs sont en plus d'être vide bien valide par rapport au pattern déjà existant et je vérifie qu'il y a aussi au moins une raison de sortie sélectionné.

@emnoze emnoze closed this Apr 15, 2020
@emnoze emnoze deleted the check_required_fields branch April 15, 2020 09:10
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.

4 participants