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
enable HTML5 e-mail validation #327
Conversation
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.
Pull request does not pass validation.
symfony/validator/4.1/manifest.json
Outdated
@@ -0,0 +1,6 @@ | |||
{ | |||
"copy-from-recipe": { | |||
"config/": "%CONFIG_DIR%/", |
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.
invalid character '}' looking for beginning of object key string
There's just one problem here: We not only need the Validator component 4.1, but the FrameworkBundle must be 4.1 or higher too. Should we solve this conflict by adding a PHP config instead which enables this based on the FrameworkBundle version? |
6a1ee89
to
420e4c1
Compare
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.
Pull request passes validation.
I think the more predictable way is to throw an exception if one uses this config with FrameworkBundle 4.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.
let's do this, and we'll see if anyone hits the reported concern (I'd doubt)
symfony/symfony#25478 (comment)