Skip to content

Better explain how to enable the validation component and its annotations #7836

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 2 commits into from

Conversation

javiereguiluz
Copy link
Member

This fixes #7797.

validation.rst Outdated
));

Besides, if you plan to use annotations to configure validation (as recommended
by the official Symfony Best Practices), replace the previous configuration by
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we add some link?

Copy link
Member Author

@javiereguiluz javiereguiluz Apr 26, 2017

Choose a reason for hiding this comment

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

I've removed this because I cannot find any reference to this in the Best Practices. So it's not a best practice but just my imagination inventing things 😁

Copy link
Contributor

@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

👍

@xabbuh
Copy link
Member

xabbuh commented Apr 27, 2017

Thank you @javiereguiluz.

xabbuh added a commit that referenced this pull request Apr 27, 2017
… its annotations (javiereguiluz)

This PR was squashed before being merged into the 2.7 branch (closes #7836).

Discussion
----------

Better explain how to enable the validation component and its annotations

This fixes #7797.

Commits
-------

dc5ff52 Better explain how to enable the validation component and its annotations
@xabbuh xabbuh closed this Apr 27, 2017
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