-
Notifications
You must be signed in to change notification settings - Fork 280
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
chore(clerk-js): Hide "Leave organization" button for last member with minimum required permissions #4299
Conversation
|
I know this is still in draft, but i wanted to be proactive here. I know these tests are hard to find so i'm resurfacing them.
|
Thanks for bringing that out @panteliselef! I'll make sure to remove those |
FYI I'm holding on this PR since we're introducing a property on FAPI organization membership in order to avoid having to itinerate through all of the memberships data during rendering time here. |
We've decided not to more forward with hiding the leave organization button due to the complexity of performing some checks on FAPI side. I'll close this PR. We'll come back to this later and revisit priorities. |
Description
Resolves ORGS-158
Checklist
npm test
runs as expected.npm run build
runs as expected.Type of change