-
Notifications
You must be signed in to change notification settings - Fork 415
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
feat: Summary of group permissions in the Project settings page #3629
feat: Summary of group permissions in the Project settings page #3629
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@fabricanva is attempting to deploy a commit to the Flagsmith Team on Vercel. A member of the Team first needs to authorize it. |
Thanks so much! Will review! |
Uffizzi Preview |
Thank you very much @fabricanva I edited the PR a little bit by componentising the permission summary, making them chips / tooltiping the remaining permissions. I'd like to use this in other areas! |
Thanks for submitting a PR! Please check the boxes below:
pre-commit
to check lintingdocs/
if required so people know about the feature!Changes
In the list of groups, the permissions that this one has were added to the row.
How did you test this code?