Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.8k
Core: Add multiple attribute to email method.#2074
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Core: Add multiple attribute to email method.#2074
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This issue/proposal has been automatically marked as idle and stale because it hasn't had any recent activity. It will be automtically closed if no further activity occurs. If you think this is wrong, or the problem still persists, just pop a reply in the comments and one of the maintainers will (try!) to follow up. |
I think this PR still has merit. |
Arkni commentedJun 16, 2018 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Hi@RobJohnston, Sorry for taking so long to review this PR. I only have one concern about this implementation. That What I would like to suggest is, we add support for passing an object to $("form").validate({rules:{field:{email:true|false|{multiple:true|false}}}}); The |
RobJohnston commentedFeb 29, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Hi@Arkni . It's now my time to apologize to you for the delay! I've been away from front-end development for some time. I didn't quite understand your comment when I first read it and not sure if I do now. From the W3, themultiple attribute is what they call aboolean attribute, but that doesn't mean that it can be set to "true" or "false". It must be multiple:true|false |
Using@Sven-Mad's method.Fixes#1722.