Skip to content
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

Integrate Globalize as one or more custom methods for number and date validation #581

Open
jzaefferer opened this issue Jan 9, 2013 · 6 comments

Comments

@jzaefferer
Copy link
Collaborator

See https://github.com/jquery/globalize/
Passing a value to the appropriate should return a good value to be considered value, while null or NaN would be considered invalid.

@johnnyreilly
Copy link
Contributor

I've been having to cater for this issue for a while now and ended up creating an extension to jQuery Validation which uses Globalize for number / date parsing. You can find it here: https://github.com/johnnyreilly/jquery-validation-globalize

I'm guessing that at some point in the future this will no longer be needed but people may find it helpful in the meantime.

@jzaefferer
Copy link
Collaborator Author

Thanks @johnnyreilly, that's great. I'll probably use that as a starting point, once Globalize is running on top of CLDR. That's currently a work in progress and may involve API changes.

@johnnyreilly
Copy link
Contributor

My pleasure @jzaefferer - I'm keeping an eye on the CLDR changes and will make any necessary amends to jquery.validate.globalize.js once API changes are clear.

@jzaefferer
Copy link
Collaborator Author

@johnnyreilly btw. the Globalize version based on CLDR is looking pretty good by now. The authors appreciate feedback you might have!

@johnnyreilly
Copy link
Contributor

Thanks for the heads up @jzaefferer - I'll try and take a look sometime soon 👍

@jzaefferer
Copy link
Collaborator Author

Note to self: Work with @johnnyreilly to update his project to Globalize 1.x, document that as an alternative to (bad) bundled methods.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants