-
Notifications
You must be signed in to change notification settings - Fork 44
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
Merge Admin / Mods Page with Groups and Members Page #10580
base: master
Are you sure you want to change the base?
Conversation
@salman-neslit Are you sure all requirements from the ticket have been fulfilled in this PR? For example, ticket mentions "We should remove one of the two from the sidebar", yet I'm still seeing Admins & Moderators as a tab in the sidebar. Seems that if there is a Manage On Chain Role Privileges functionality in Members & Groups, then the Admins & Moderators page should be removed. |
...ient/scripts/views/pages/CommunityGroupsAndMembers/Members/MembersSection/MembersSection.tsx
Show resolved
Hide resolved
@Israellund I also left the note above regarding the removal of previous funcnality will be handled in the follow up PR. as there is another issue that also needs to be addressed Issue#9521 these 2 are linked |
Works well, few things we should handle tho
Screen.Recording.2025-01-30.at.7.15.22.PM.mov |
Link to Issue
Closes: #9978
Description of Changes
"How We Fixed It"
Test Plan
Deployment Plan
Other Considerations