-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Validator] Add Validation::createCallable() #15600
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
Conversation
Hey! Oh no, it looks like you have made this PR towards a branch that is not maintained anymore. :/ Cheers! Carsonbot |
d65b84e
to
1539bca
Compare
@carsonbot Thank you for the hint. I rebased against 5.3 |
Hi @W0rma! This is a great contribution, this feature was missing docs for quite some time. I have some suggestions on the location/content of this article. Please let me know if you don't agree or if you don't have the time to make the changes, I'm sure we can get this merged in any case :)
|
1539bca
to
b89e08c
Compare
b89e08c
to
6cc9f38
Compare
@wouterj Thank you for the suggestions. I think they are all very good ideas and therefore I applied them to this PR. |
@W0rma @javiereguiluz I just stumbled over this in the docs and noticed that the mentioned method |
I just took the examples from https://symfony.com/blog/new-in-symfony-5-1-validator-improvements#validation-callables and #15093
Fixes #13045
Fixes #15093