-
Notifications
You must be signed in to change notification settings - Fork 38
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
Undocumented/access-management #1959
Conversation
Undocumented/render tables
…mgmt restructure codes, remove RoleLists and RoleTables
Undocumented/table buttons
…s-mgmt-add-person
Undocumented/access mgmt add person
Undocumented/access mgmt refactor
…etkinMembership type
Undocumented/add inherited logic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work on this team! The feature is working great. I have tried the preview build and also looked at the code. I found some potential bugs in the code, but mostly I have a bunch of thoughts about naming.
Maybe @kaulfield23 can take a look at this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! I found a minor thing that I think we should improve before merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! This looks ready to merge to me. 💯
Description
This PR implements Settings page and its necessary functionalities. User can view the 'Administrator' / 'Organizer' list and demote, promote and remove official.
Screenshots
Mozilla.Firefox.2024-05-21.16-30-57.mp4
Changes
OfficialList
,RoleAddPersonButton
useRoles
,useRolesMutations
SettingsLayout
Notes to reviewer
Related issues
Undocumented