8000 Mention security voters as the recommended alternative to ACL by javiereguiluz · Pull Request #8719 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

Mention security voters as the recommended alternative to ACL #8719

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 3 commits into from
Nov 23, 2017

Conversation

javiereguiluz
Copy link
Member

This morning we had this conversation with a person who thinks that Symfony Docs are bad: https://twitter.com/PhillippOh/status/932905608998805504 At the end, it was all about thinking that the rest of Symfony Docs are the same as the poor ACL docs. So, let's recommend using voters everywhere.

Reowrds are welcome because I don't like the repetition of the word "alternative". Thanks!

@weaverryan
Copy link
Member

Status: Reviewed

@xabbuh
Copy link
Member
xabbuh commented Nov 23, 2017

Thank you Javier.

@xabbuh xabbuh merged commit 6e8f763 into symfony:master Nov 23, 2017
xabbuh added a commit that referenced this pull request Nov 23, 2017
… ACL (javiereguiluz, weaverryan)

This PR was merged into the master branch.

Discussion
----------

Mention security voters as the recommended alternative to ACL

This morning we had this conversation with a person who thinks that Symfony Docs are bad: https://twitter.com/PhillippOh/status/932905608998805504  At the end, it was all about thinking that the rest of Symfony Docs are the same as the poor ACL docs. So, let's recommend using voters everywhere.

Reowrds are welcome because I don't like the repetition of the word *"alternative"*. Thanks!

Commits
-------

6e8f763 tweak
0941a49 tweak
4747306 Mention security voters as the recommended alternative to ACL
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
0