-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
[Validator] Add docs for bridge twig validator #20836 #20863
Conversation
bridge/twig/validation.rst
Outdated
Twig template: | ||
|
||
.. code-block:: php |
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.
Twig template: | |
.. code-block:: php | |
Twig template:: |
bridge/twig/validation.rst
Outdated
set this to `false`. | ||
|
||
.. code-block:: php |
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.
set this to `false`. | |
.. code-block:: php | |
set this to `false`:: |
90b3c8e
to
7f3faf7
Compare
7f3faf7
to
ac9e20c
Compare
@OskarStark done, but I think this #20872 needs to be merged first |
@sfmok thanks a lot for contributing these docs! Your contribution was perfectly correct, but we changed some things while merging. The fact that this constraint is stored in the Twig Bridge is an "implementation detail" that most users don't care about. We have other similar constraints (e.g. So, we made some changes (see 344d8ba) to remove the new Also, even if the constraint page titles only use the name of the constraint (e.g. just Thanks! |
Got it, that makes perfect sense. Thanks! |
This pull request adds documentation for the new Twig Validator constraint introduced in the Twig Bridge.
Changes:
message
andskipDeprecations
.index.rst
to include the new page under the "Topics" section for the Twig Bridge.Related Issue:
Issue #20836 - Add docs for Twig validator
Related PR:
Symfony PR #58805 – Introduced the Twig validator constraint in the Twig Bridge.