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

Multiple email address validation #1812

Closed
dolphinlin opened this issue May 20, 2019 · 2 comments
Closed

Multiple email address validation #1812

dolphinlin opened this issue May 20, 2019 · 2 comments
Assignees
Labels
feature New functionality or improvement
Milestone

Comments

@dolphinlin
Copy link
Contributor

dolphinlin commented May 20, 2019

Describe the problem you are trying to fix (provide as much context as possible)

Try to validate more than one email in a string that separate by comma, it throw exception.
Now for most part, the multiple email address is probably supported.
EX. abc@aaa.com, bbb@ccc.com

Which API (or modification of the current API) do you suggest to solve that problem ?

joi.string().email()

// maybe
joi.string().email({
    multiple: true,
})

Are you ready to work on a pull request if your suggestion is accepted ?

Yes.

@hueniverse
Copy link
Contributor

Are you going to consume this data as a string later? Or will split that string again?

@dolphinlin
Copy link
Contributor Author

nope!
I use the multiple email address to send by Mailgun service.
https://documentation.mailgun.com/en/latest/api-sending.html#sending

@hueniverse hueniverse added this to the 16.0.0 milestone Jun 14, 2019
@hueniverse hueniverse self-assigned this Jun 14, 2019
@hueniverse hueniverse added the v16 label Aug 10, 2019
@hueniverse hueniverse added feature New functionality or improvement and removed request labels Sep 19, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Mar 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature New functionality or improvement
Projects
None yet
Development

No branches or pull requests

2 participants