-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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] Document the new NoSuspiciousCharacters
constraint
#17897
[Validator] Document the new NoSuspiciousCharacters
constraint
#17897
Conversation
edba633
to
40c94ec
Compare
05f566e
to
2792c27
Compare
47d2e1b
to
22b3e54
Compare
…to validate a string is not suspicious (MatTheCat) This PR was merged into the 6.3 branch. Discussion ---------- [Validator] Add a `NoSuspiciousCharacters` constraint to validate a string is not suspicious | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | Fix #49268 | License | MIT | Doc PR | symfony/symfony-docs#17897 Leverage `Spoofchecker::isSuspicious` in a new constraint. Commits ------- 12f901f [Validator] Add a `NoSuspiciousCharacters` constraint to validate a string is not a spoof attempt
…to validate a string is not suspicious (MatTheCat) This PR was merged into the 6.3 branch. Discussion ---------- [Validator] Add a `NoSuspiciousCharacters` constraint to validate a string is not suspicious | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | Fix #49268 | License | MIT | Doc PR | symfony/symfony-docs#17897 Leverage `Spoofchecker::isSuspicious` in a new constraint. Commits ------- 12f901f438 [Validator] Add a `NoSuspiciousCharacters` constraint to validate a string is not a spoof attempt
…to validate a string is not suspicious (MatTheCat) This PR was merged into the 6.3 branch. Discussion ---------- [Validator] Add a `NoSuspiciousCharacters` constraint to validate a string is not suspicious | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | Fix #49268 | License | MIT | Doc PR | symfony/symfony-docs#17897 Leverage `Spoofchecker::isSuspicious` in a new constraint. Commits ------- 12f901f438 [Validator] Add a `NoSuspiciousCharacters` constraint to validate a string is not a spoof attempt
22b3e54
to
019c23f
Compare
The feature got merged 🎉 |
|
||
The ``NoSuspiciousCharacters`` constraint was introduced in Symfony 6.3. | ||
|
||
.. |
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.
This is a comment, and therefore not rendered on the website, am I correct @wouterj @javiereguiluz ?
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.
I hope so! I was following https://docutils.sourceforge.io/docs/ref/rst/restructuredtext.html#block-quotes
Empty comments may be used to explicitly terminate preceding constructs that would otherwise consume a block quote
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.
What Oskar meant is that this content is a comment that won't be rendered on symfony.com. I've reworded it a bit to add it to the normal text so people can read it.
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.
Should also be added to reference/constraints.rst
❗
Done, thanks for the heads-up |
a35e0c8
to
69a1ddd
Compare
69a1ddd
to
8453ee2
Compare
NoSuspiciousCharacters
constraintNoSuspiciousCharacters
constraint
8453ee2
to
e4d1b82
Compare
Mathieu, this was a fantastic contribution and a great feature! Thanks a lot for contributing it. |
…to validate a string is not suspicious (MatTheCat) This PR was merged into the 6.3 branch. Discussion ---------- [Validator] Add a `NoSuspiciousCharacters` constraint to validate a string is not suspicious | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | Fix #49268 | License | MIT | Doc PR | symfony/symfony-docs#17897 Leverage `Spoofchecker::isSuspicious` in a new constraint. Commits ------- 12f901f438 [Validator] Add a `NoSuspiciousCharacters` constraint to validate a string is not a spoof attempt
Feature PR: symfony/symfony#49300