Skip to content
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

🎁 i445 remove roles #920

Merged
merged 2 commits into from
Dec 7, 2023
Merged

🎁 i445 remove roles #920

merged 2 commits into from
Dec 7, 2023

Conversation

kirkkwang
Copy link
Contributor

@kirkkwang kirkkwang commented Dec 5, 2023

Story

Add ability to remove roles

6957b20

This commit will add the ability for user_managers to remove site roles
from users other than admin. Admins will be able to remove any site
role from any user.

Ref:

Update translations

61da009

This commit will run the i18n script to update the translations for the
new roles.

Screenshots / Video

Screen.Recording.2023-12-06.at.16.16.44.mp4

Notes

@kirkkwang kirkkwang force-pushed the i445-remove-roles branch 4 times, most recently from e9f3515 to 7d56219 Compare December 6, 2023 23:10
@kirkkwang kirkkwang changed the title wip 🎁 wip Dec 6, 2023
This commit will add the ability for user_managers to remove site roles
from users other than admin.  Admins will be able to remove any site
role from any user.

Ref:
  - #445
This commit will run the i18n script to update the translations for the
new roles.
@kirkkwang kirkkwang changed the title 🎁 wip 🎁 i445 remove roles Dec 7, 2023
@kirkkwang kirkkwang marked this pull request as ready for review December 7, 2023 00:19
@kirkkwang kirkkwang merged commit ef0af1f into main Dec 7, 2023
6 of 7 checks passed
@kirkkwang kirkkwang deleted the i445-remove-roles branch December 7, 2023 15:09
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.

2 participants