christianalfoni/formsy-react

Incompatible API changes

Open

#330 ouverte le 22 avr. 2016

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)JavaScript (423 forks)github user discovery
help wantedquestion

Métriques du dépôt

Stars
 (2 574 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

My suggestions:

  1. Remove automatic validation at start rendering and in setValue.
  2. Rename validate to customValidate.
  3. Add validate method to Form and Mixin for manually validation.

This should make it very easy following scenarios:

  • validate on change
  • validate on blur
  • validate on submit
  • ...

@twisty @mbrookes and other Your opinion is important

Guide contributeur

Direction de recherche
Lisez la documentation de l'API liée et évaluez l'impact du renommage de `validate` et de la suppression de la validation automatique. Considérez comment ces changements affectent les modèles de validation on change, on blur et on submit.
Stack technique
react
Domaine
frontend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
ReactJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75