Page MenuHomePhabricator

[web] Show option to change role from the members modal
ClosedPublic

Authored by rohan on Jun 7 2023, 4:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 19 2024, 2:30 PM
Unknown Object (File)
Feb 19 2024, 12:55 PM
Unknown Object (File)
Feb 19 2024, 12:53 PM
Unknown Object (File)
Feb 19 2024, 12:44 PM
Unknown Object (File)
Feb 19 2024, 11:32 AM
Unknown Object (File)
Feb 19 2024, 3:31 AM
Unknown Object (File)
Jan 11 2024, 5:03 PM
Unknown Object (File)
Dec 21 2023, 2:13 PM

Details

Summary

As a first step in allowing admins to change member roles, the member panel option needed to be refactored to a more general approach to 'Change Role', as opposed to just toggling an admin status.

https://linear.app/comm/issue/ENG-4037/show-option-to-change-role-from-the-members-modal

Test Plan

Confirm that the dropdown shows the updated icon and text

Screenshot 2023-06-07 at 8.13.20 AM.png (1×1 px, 99 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rohan requested review of this revision.Jun 7 2023, 5:16 AM
atul added a subscriber: ted.

Don't think the plus-circle icon makes sense for "Change Role," but defer to @ted on that.

web/modals/threads/members/member.react.js
77 ↗(On Diff #27512)

Are we sure plus-circle is an appropriate icon for "Change Role"?

CC @ted for his thoughts

This revision is now accepted and ready to land.Jun 7 2023, 12:53 PM
web/modals/threads/members/member.react.js
77 ↗(On Diff #27512)

The plus-circle seems to be used in the designs (screenshot below is from Figma) so I used that one. Happy to change though

Screenshot 2023-06-07 at 4.12.52 PM.png (1×796 px, 133 KB)

Remove memberIsAdmin import

web/modals/threads/members/member.react.js
77 ↗(On Diff #27512)

I agree with @atul. @ted doesn't seem to have responded here... pinging him again

web/modals/threads/members/member.react.js
77 ↗(On Diff #27512)

Still no response from @ted@rohan can you make sure to ping him about this? It would be good to create a Linear task to track