-
Notifications
You must be signed in to change notification settings - Fork 1
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
Validation should be pluggable #31
Comments
Also, the forms validation api should be updated to pass in more -- On Mon, Aug 31, 2015 at 2:49 PM, gevou [email protected] wrote:
|
Let's take this in the direction of while we provide a set of validations (that we can separate into another package), let's make a flexible enough api for people to easily plug in their own validation, or at least easily create a connector between something like Astronomy's validations and the forms api so they can choose how to validate without again writing a lot of duplicate validations or boilerplate. |
any updates on validation? Or some tips how to roll my own? maybe with the {{error}} |
Built-in validators in validators.js have been built based on a previous version of the Forms API which passed all the parameters to a validator function as a single
context
object. (see line 1 of validators.js)They should be updated in order to reflect the new API and allow the following usage syntax within a schema:
carType: Forms.validators.oneOf([ 'sport', 'sedan', ... ])
The text was updated successfully, but these errors were encountered: