christianalfoni/formsy-react

Incompatible API changes

Open

#330 aperta il 22 apr 2016

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)JavaScript (423 fork)github user discovery
help wantedquestion

Metriche repository

Star
 (2574 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Leggi la documentazione API collegata e valuta l'impatto della ridenominazione di `validate` e della rimozione della convalida automatica. Considera come questi cambiamenti influenzano i pattern di validazione su onChange, onBlur e onSubmit.
Tech stack
react
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ReactJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75