Skip to content

[Validator] replace expressionLanguageSyntax with new expressionSyntax #17555

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

MrYamous
Copy link
Contributor

@MrYamous MrYamous commented Dec 12, 2022

As expressionSyntax deprecates expressionLanguageSyntax, replace it from constraint's list

Related code PR

@OskarStark
Copy link
Contributor

Can you please link the code PR in the PR header? Thanks

@@ -16,7 +16,7 @@ String Constraints
~~~~~~~~~~~~~~~~~~

* :doc:`Email </reference/constraints/Email>`
* :doc:`ExpressionLanguageSyntax </reference/constraints/ExpressionLanguageSyntax>`
* :doc:`ExpressionSyntax </reference/constraints/ExpressionSyntax>`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is another file where it needs to be renamed, please search through the repository, it looks almost the same

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I couldn't find other occurrences while merging. Maybe this was fixed and push-forced into this PR.

@MrYamous MrYamous force-pushed the validator/expressionsyntax-replace-expressionlanguagesyntax branch from 125c660 to 3a35ff2 Compare December 12, 2022 14:03
@javiereguiluz javiereguiluz merged commit d473b13 into symfony:6.1 Dec 13, 2022
@javiereguiluz
Copy link
Member

Thanks Matthieu!

@MrYamous MrYamous deleted the validator/expressionsyntax-replace-expressionlanguagesyntax branch March 8, 2023 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants