-
Notifications
You must be signed in to change notification settings - Fork 411
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
docs: Deprecate group-level permissions. Rewrite RBAC docs and add Mermaid diagrams #4469
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Skipped Deployments
|
Docker builds report
|
Uffizzi Preview |
7f55028
to
d17b081
Compare
Reminder before merging: add the actual Flagsmith version when this is released here https://github.com/Flagsmith/flagsmith/pull/4469/files#diff-a272079eeec1855f9514e0c2254805b5cc9dcc82b3e09f6e5d4a26cda188c4b1R123 |
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.
Excellent work @rolodato. Love the initiative here, and the docs are very nice and clear!
Deprecates group-level permissions. This is only a UI change and no action is required from customers. Added
target=_blank
to all links in the deprecation notice to avoid accidentally discarding any unsaved changes.Can probably make more improvements to the docs here, but I'd be happy to merge it as-is.
The part about provisioning users and permissions is not strictly related to RBAC so it could go in its own document later on. This is probably the best place for it to be at the moment.
permissions.mov