Skip to content

Additions to PR 263#265

Merged
arogachev merged 7 commits intomasterfrom
264-access-check-default-roles
Apr 23, 2024
Merged

Additions to PR 263#265
arogachev merged 7 commits intomasterfrom
264-access-check-default-roles

Conversation

@arogachev
Copy link
Contributor

@arogachev arogachev commented Apr 22, 2024

Q A
Is bugfix? ✔️
New feature?
Breaks BC?

@arogachev arogachev linked an issue Apr 22, 2024 that may be closed by this pull request
@arogachev arogachev self-assigned this Apr 22, 2024
@arogachev arogachev added the type:bug Bug label Apr 22, 2024
@arogachev arogachev added this to the 2.1 milestone Apr 22, 2024
@arogachev arogachev marked this pull request as ready for review April 22, 2024 12:57
@arogachev arogachev requested a review from a team April 22, 2024 12:58
@arogachev arogachev added the status:code review The pull request needs review. label Apr 22, 2024
@arogachev arogachev merged commit 1b8bbbb into master Apr 23, 2024
@arogachev arogachev deleted the 264-access-check-default-roles branch April 23, 2024 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review. type:bug Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Manager::userHasPermission() - default roles are not checked

3 participants