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

Ability to pass blacklisted characters for isEmail()? #654

Closed
grelas opened this issue Apr 19, 2017 · 9 comments
Closed

Ability to pass blacklisted characters for isEmail()? #654

grelas opened this issue Apr 19, 2017 · 9 comments

Comments

@grelas
Copy link

grelas commented Apr 19, 2017

For the app I'm building, we do not allow single quotes ' in the user's email address (same reasoning from this SO post). Although according to the spec, single quotes are allowed, is there a way to not allow that specific character?

@chriso
Copy link
Collaborator

chriso commented Jun 22, 2017

It's not currently possible to whitelist or blacklist characters in the local part of the email address. This would be a trivial addition though I imagine. Happy to accept a PR 😄

@sudhir512kj
Copy link

@profnandaa I would like to work on it.

@profnandaa
Copy link
Member

@sudhir512kj -- sure, go for it! 👍

@mmena1
Copy link
Contributor

mmena1 commented Oct 24, 2019

@chriso @profnandaa Is this issue still valid? If so, can you explain what would be the acceptance criteria?

@raxraj
Copy link

raxraj commented Oct 5, 2020

Is this issue still open? I want to contribute to this.

@profnandaa
Copy link
Member

profnandaa commented Oct 5, 2020

Sure, it's in review - 1414 / you are welcome to contribute on another, looking forward :)

@profnandaa
Copy link
Member

Sorry #1449

@raxraj
Copy link

raxraj commented Oct 5, 2020

Oh I see thanks @profnandaa

@tux-tn
Copy link
Member

tux-tn commented Apr 25, 2021

Closed in #1449

@tux-tn tux-tn closed this as completed Apr 25, 2021
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

7 participants