search

NewOldMax / react-form-validator-core

Core validator component for react forms
MIT License
95 stars 44 forks source link

fix rules bug: the 'required' validator was always returning 'true' #7

Closed 7Flow closed 6 years ago

7Flow commented 6 years ago

There was no 'else' when the validator handles the case of a validator with 'extra' (supplied parameters to the rules function) It makes the 'required' validator makes no call to the rule 'required'.

NewOldMax commented 6 years ago

Sorry @7Flow, but I must reject this PR. It will break all current forms with unnecessary instant required validation. I have plan to allow users enable this validation, if they want