Skip to content

Update Custom Constraints documentation #11060

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

Closed
wants to merge 1 commit into from

Conversation

MendicantBias90
Copy link

In the Class Constraint Validator section, the example that shows how to apply the custom validator to the whole class it's wrong, it applies the ConstraintValidator class instead of the Constraint class to validation.

In the Class Constraint Validator section, the example that shows how to apply the custom validator to the whole class it's wrong, it applies the ConstraintValidator class instead of the Constraint class to validation.
@javiereguiluz
Copy link
Member

@MendicantBias90 thanks a lot for your contribution. I was going to merge it but when reviewing the changes they felt familiar to me. The reason is that we already merged these changes in a different pull request: #11180.

I'm really sorry because your pull request was submitted earlier and we should have merged yours. I'm sorry because it was my fault. This is not common, but it sometimes happens when we receive lots of contributions.

We're looking forward to receive more contributions from you in the future. Thanks!

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.

3 participants