-
-
Notifications
You must be signed in to change notification settings - Fork 172
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
Number validator? #60
Comments
My goal was to be able to replace the number validator with the inclusion/exclusion validators since they are type checked. If this isnt enough, just use a custom validator for now. If there is a need for a number validator we can create one later down the line. |
Yep, I've created my own number validator for now. But yea, it would be great to have one built in, it's rather common. |
I initially thought this can be replaced by type checked inclusion/exclusion but there are way too many specific use cases. This should also suffice #60
Please upgrade to 2.1.0 and check out the number validator docs 😸 |
I can't locate number validator neither in code nor in the docs. I see there is message for that https://github.com/offirgolan/ember-cp-validations/blob/master/addon/validators/messages.js#L78. Should I create custom one then?
The text was updated successfully, but these errors were encountered: