Skip to content
This repository has been archived by the owner on Aug 15, 2019. It is now read-only.

Add email validation option, set to false by default #227

Closed
wants to merge 1 commit into from
Closed

Add email validation option, set to false by default #227

wants to merge 1 commit into from

Conversation

martincarlin87
Copy link

Added an option validateEmails as I was using this for a CC list and didn't want any tags to be added unless it was a valid email address.

@lucknerjb
Copy link
Contributor

Thanks for the PR however, a few things:

1 - There are too many whitespace diffs for this to be merged in
2 - validateEmails is too specific and is more of a use-case than an actual needed feature

This kind of validation can be added in the onAddTag callback.

At best, adding a validations key to the options would be an option but even that might be overkill.

@lucknerjb lucknerjb closed this May 28, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants