Skip to content

Store voters in Security #42

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 1 commit into from
May 1, 2017
Merged

Store voters in Security #42

merged 1 commit into from
May 1, 2017

Conversation

GuilhemN
Copy link
Contributor

As caught by @xabbuh, in the docs, voters are stored in Security, not in Voter.

Copy link
Member

@xabbuh xabbuh left a comment

Choose a reason for hiding this comment

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

👍

@fabpot
Copy link
Member

fabpot commented May 1, 2017

@GuilhemN Can you do another PR to add the comments you added in Symfony SE? I think they are quite useful.

@fabpot
Copy link
Member

fabpot commented May 1, 2017

Thank you @GuilhemN.

@fabpot fabpot merged commit 6185425 into symfony:master May 1, 2017
fabpot added a commit that referenced this pull request May 1, 2017
This PR was merged into the master branch.

Discussion
----------

Store voters in Security

[As caught by @xabbuh](symfony/symfony-standard#1064 (comment)), in the docs, voters are stored in `Security`, not in `Voter`.

Commits
-------

6185425 Store voters in Security
@GuilhemN GuilhemN deleted the patch-1 branch May 1, 2017 17:31
@GuilhemN GuilhemN mentioned this pull request May 1, 2017
@GuilhemN
Copy link
Contributor Author

GuilhemN commented May 1, 2017

sure, see #43

fabpot added a commit that referenced this pull request May 1, 2017
This PR was merged into the master branch.

Discussion
----------

Add comments

Related to #42 and symfony/symfony-standard#1064

Commits
-------

52c22bd Add comments
VolCh pushed a commit to VolCh/recipes that referenced this pull request Nov 30, 2017
markovlatkovic pushed a commit to markovlatkovic/recipes that referenced this pull request May 24, 2024
This PR was merged into the master branch.

Discussion
----------

Add comments

Related to symfony/recipes#42 and symfony/symfony-standard#1064

Commits
-------

52c22bd Add comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants