-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
TEST PERFORMED BY NATE - 10/8/23
|
Each adminRole belongs to a set of undocumented strings, not including "admin". "meetings" and "charters" have been used instead.
I owe you a much more extensive review; very sorry about that! So far, looking great! |
Amazing! Looks like you fixed all of those bugs on the frontend! |
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.
Well done! Just going to get rid of a few Prettier-inserted spaces (ick), then merge!
{org.membership?.adminPrivileges && ( | ||
<TabItem label={"Admin Panel"} exact={false} to={match.path + "/admin"} icon={<Settings />} /> | ||
)} | ||
{user?.adminRoles?.some(s => s.role === "admin") && ( | ||
<FlexCenter> | ||
<Button onClick={CopyOrgId}> |
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.
Note to self, test and verify these buttons, so far looking great
Enhancement for Copy ID #722 (set to admin only)
Auto Update Members for approving/rejecting members as well as removing one. #725 and #717
Also fixes #190 with a loading sign
ALSO fixes meetings not auto updating