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

Allow business only email address #1639

Closed
imjul1an opened this issue Apr 10, 2021 · 8 comments · Fixed by #1641
Closed

Allow business only email address #1639

imjul1an opened this issue Apr 10, 2021 · 8 comments · Fixed by #1641

Comments

@imjul1an
Copy link

Describe the bug
Allow business only email address.

Examples
[email protected] - valid
[email protected] - invalid

@imjul1an
Copy link
Author

it's a feature request, not a bug.

@fedeci
Copy link
Contributor

fedeci commented Apr 10, 2021

What would be the standard to distinguish between a "business email" and a "non-business" one?

@imjul1an
Copy link
Author

@fedeci I would include most popular email domains into the ignore list.

const ignore_public_email_domains = ['gmail.com', 'yahoo.com', 'hotmail.com', 'aol.com', ...]

validator.isBusinessEmail('[email protected]') // false
validator.isBusinessEmail('[email protected]') // true

@fedeci
Copy link
Contributor

fedeci commented Apr 10, 2021

We may add an option to let the end user to exclude specific domains, but I would not add some default ones.

@imjul1an
Copy link
Author

imjul1an commented Apr 10, 2021

This sounds like a very good option. However, I'd also include a list of most popular domains, so that users can include the pre-default list of excluded domains.

@imjul1an
Copy link
Author

You know, you are right! The option you mentioned would be more than enough.

@fedeci
Copy link
Contributor

fedeci commented Apr 10, 2021

I am working on this.

@ezkemboi
Copy link
Member

If it more of business items, I did research and found out that https://hunter.io/email-verifier can offer a better validation.

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

Successfully merging a pull request may close this issue.

5 participants