Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Mailer] Add Support for email-validator v3#39657
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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
michaljusiega commentedDec 29, 2020
Ticket:#39656 |
OskarStark commentedDec 29, 2020
Can please check if other Components use this too and bump them also? Thanks |
| "require": { | ||
| "php":">=7.2.5", | ||
| "egulias/email-validator":"^2.1.10", | ||
| "egulias/email-validator":"^2.1.10|^3.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
There are more places where this package needs to be bumped. Furthermore, we should try to patch the 4.4 branch instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
yes, on all the mailer components
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I would prefer 4.4, too
derrabus commentedJan 2, 2021
Closing in favor of#39685. |
Uh oh!
There was an error while loading.Please reload this page.