Skip to content

Added documentation for the priority access decision strategy #12956

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
Merged
merged 1 commit into from
Feb 5, 2020
Merged

Added documentation for the priority access decision strategy #12956

merged 1 commit into from
Feb 5, 2020

Conversation

aschempp
Copy link
Contributor

fixes #12830

@HeahDude HeahDude added this to the 5.1 milestone Jan 18, 2020
HeahDude
HeahDude suggested changes Jan 18, 2020
View reviewed changes
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.

Thanks for your contributions., looks good!

@aschempp aschempp requested a review from HeahDude January 20, 2020 11:02
@aschempp
Copy link
Contributor Author

I have added the versionadded directive

OskarStark
OskarStark reviewed Jan 20, 2020
View reviewed changes
@OskarStark OskarStark force-pushed the feature/security-priority-strategy branch from 42d57a9 to c903f4a Compare February 5, 2020 05:58
@OskarStark
Copy link
Contributor

OskarStark commented Feb 5, 2020 •
edited
Loading

Thanks for your work on this new feature! I made some adjustments while merging: 6113a66

@OskarStark OskarStark closed this in 2754e72 Feb 5, 2020
@OskarStark OskarStark merged commit c903f4a into symfony:master Feb 5, 2020
OskarStark added a commit that referenced this pull request Feb 5, 2020
@OskarStark
minor. refs #12956
6113a66
@aschempp aschempp deleted the feature/security-priority-strategy branch February 7, 2020 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OskarStark OskarStark OskarStark approved these changes

@HeahDude HeahDude HeahDude approved these changes

Assignees
No one assigned
Labels
Security Status: Reviewed
Projects
None yet
Milestone
5.1
Development

Successfully merging this pull request may close these issues.

Added access decision strategy to respect voter priority
4 participants
@aschempp @OskarStark @HeahDude @carsonbot