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

Modification de la boite modale de publication d'un contenu #4820

Closed
wants to merge 6 commits into
base: dev
from

Conversation

Projects
None yet
5 participants
@nils-van-zuijlen
Contributor

nils-van-zuijlen commented Dec 22, 2017

Ajout de la gestion des erreurs de formulaire HTML5 pour les formulaires de validation.
Correction de l'affichage des erreurs levées par le serveur.

Numéro du ticket concerné: #4817

Contrôle qualité

  • Créez un nouveau contenu
  • Demandez sa validation
  • Essayez de le valider avec un commentaire de publication plus court que 3 caractères, le navigateur doit refuser l'envoi du formulaire.
  • À l'aide des outils développeur du navigateur, retirer l'attribut min-length="3" de la zone de texte
  • Envoyez le formulaire avec des espaces ou moins de 3 caractères, le site doit afficher l'erreur Vous devez fournir un commentaire aux validateurs. ou l'erreur Votre commentaire doit faire au moins 3 caractères., sans les entourer de [' et ']

TODO:

  • attribut min-length
  • simplification du code de validation de formulaire
  • affichage correct des erreurs
  • rédaction des tests unitaires

nils-van-zuijlen added some commits Dec 21, 2017

Simplified handling of errors
In form zds.tutorialv2.forms.AcceptValidationForm
Using django.forms.Forms.add_error

@zestedesavoir zestedesavoir deleted a comment from coveralls Dec 23, 2017

@Situphen Situphen changed the title from Modification de la boite modale de publication d'un contenu to [WIP] Modification de la boite modale de publication d'un contenu Dec 23, 2017

@Situphen

This comment has been minimized.

Show comment
Hide comment
@Situphen

Situphen Dec 23, 2017

Contributor

Merci pour cette PR :)

Je me suis permis d'ajouter WIP (Work In Progress) dans le titre pour préciser que ta PR n'est pas finie :)

Contributor

Situphen commented Dec 23, 2017

Merci pour cette PR :)

Je me suis permis d'ajouter WIP (Work In Progress) dans le titre pour préciser que ta PR n'est pas finie :)

@Situphen Situphen added the C-Back label Dec 23, 2017

nils-van-zuijlen added some commits Jan 9, 2018

Show outdated Hide outdated templates/base.html Outdated
Écriture des tests
- Tests du tag messages
- Optimisation des imports
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jan 14, 2018

Coverage Status

Coverage decreased (-5.3%) to 71.852% when pulling 738c70e on nils-van-zuijlen:fix-4817 into 28764a9 on zestedesavoir:dev.

coveralls commented Jan 14, 2018

Coverage Status

Coverage decreased (-5.3%) to 71.852% when pulling 738c70e on nils-van-zuijlen:fix-4817 into 28764a9 on zestedesavoir:dev.

@nils-van-zuijlen nils-van-zuijlen changed the title from [WIP] Modification de la boite modale de publication d'un contenu to Modification de la boite modale de publication d'un contenu Jan 14, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

@zestedesavoir zestedesavoir deleted a comment from coveralls Jan 19, 2018

pierre-24 added a commit to pierre-24/zds-site that referenced this pull request Aug 3, 2018

@pierre-24

This comment has been minimized.

Show comment
Hide comment
@pierre-24

pierre-24 Aug 3, 2018

Member

Reprise dans #4993, merci à toi :)

Member

pierre-24 commented Aug 3, 2018

Reprise dans #4993, merci à toi :)

@pierre-24 pierre-24 closed this Aug 3, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment