Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Validator] Add pattern in Regex constraint violations #17436

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

Merged
merged 1 commit into from
Dec 9, 2022

Conversation

alamirault
Copy link
Contributor

@alamirault alamirault commented Nov 17, 2022

For 6.3

symfony/symfony#48232

Waiting Code merge (Is there a tip in order to set label at PR creation ?)

@94noni
Copy link
Contributor

94noni commented Nov 18, 2022

Is there a tip in order to set label at PR creation ?

Perhaps create a new carsonbot action that detect the symfony/symfony PR number, and check its state if merged or not ^^

@OskarStark
Copy link
Contributor

We created the 6.3 branch, can you please retarget your PR? Thanks

@OskarStark OskarStark changed the title [Validator] Add pattern in Regex constraint violations [Validator] Add pattern in Regex constraint violations Nov 28, 2022
@alamirault alamirault force-pushed the feature/regex-pattern-violation branch from 4ae4534 to db15323 Compare November 28, 2022 20:07
@alamirault alamirault changed the base branch from 6.2 to 6.3 November 28, 2022 20:07
@alamirault
Copy link
Contributor Author

@OskarStark I rebased on 6.3

fabpot added a commit to symfony/symfony that referenced this pull request Dec 9, 2022
…olations (alamirault)

This PR was merged into the 6.3 branch.

Discussion
----------

[Validator] Add `{{pattern}}` to `Regex` constraint violations

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License       | MIT
| Doc PR        | symfony/symfony-docs/pull/17436

This PR add parameter `pattern` to `Regex` constraints violations.
This is useful for building message to technical users/consumers. (like limit, choices, min, max ... in other constraints )

Commits
-------

369b051 [Validator] Add pattern in Regex constraint violations
@OskarStark OskarStark removed the Waiting Code Merge Docs for features pending to be merged label Dec 9, 2022
@OskarStark
Copy link
Contributor

It took time, but here we go, this is in now. Thank you very much Antoine.

@OskarStark OskarStark merged commit 5322f64 into symfony:6.3 Dec 9, 2022
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