-
Notifications
You must be signed in to change notification settings - Fork 411
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
A role without a tag should not have its permissions restricted. #4288
Comments
@novakzaballa the example you have given in the description is not valid. The permission for deleting features is the |
@matthewelwell I'm sorry, there was confusion in the PR description. I will correct this with the correct permission and reopen this issue |
Related with #1535 |
@novakzaballa I think this is a duplicate of #4506 and was fixed here: https://github.com/Flagsmith/flagsmith-rbac/pull/24? Is that correct? I am not able to reproduce this |
Ah, I am able to reproduce this if I tag the feature? |
Hey @gagantrivedi, you’re right. The PR also resolved part of the issue with the roles in the same way. I’ll merge it along with my PR. |
Fixed in #4523 |
How are you running Flagsmith
Describe the bug
If a role without a tag should not have its permissions(
UPDATE_FEATURE_STATE
and/orDELETE_FEATURE
) restricted.Steps To Reproduce
Expected behavior
If there is no tag restricting the permissions, the feature should be deleted successfully.
Screenshots
No response
CC @matthewelwell @gagantrivedi @kyle-ssg
The text was updated successfully, but these errors were encountered: