Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update custom_constraint.rst #6634

Closed
wants to merge 1 commit into from
Closed

Update custom_constraint.rst #6634

wants to merge 1 commit into from

Conversation

axelvnk
Copy link

@axelvnk axelvnk commented Jun 2, 2016

Clear confusion about defining a validator as a service and using it with a custom constraint

Update custom_constraint.rst
35c716e 
Clear confusion about defining a validator as a service and using it with a custom constraint
weaverryan added a commit that referenced this pull request Jul 10, 2016
@weaverryan
bug #6634 Update custom_constraint.rst (axelvnk)
80594da 
This PR was submitted for the 3.1 branch but it was merged into the 2.7 branch instead (closes #6634).

Discussion
----------

Update custom_constraint.rst

Clear confusion about defining a validator as a service and using it with a custom constraint

Commits
-------

6ae9022 Update custom_constraint.rst
@weaverryan
Copy link
Member

Thanks @axelvnk! I've just merged this in, and actually - noticed that this is in worse shape than I thought :). Check out #6746 - a more recent change in Symfony allows us to get rid of this alias business entirely.

Cheers!

@weaverryan weaverryan closed this Jul 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@axelvnk @weaverryan