Skip to content
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

New RBAC UI #14577

Merged
merged 121 commits into from
Oct 23, 2024
Merged

New RBAC UI #14577

merged 121 commits into from
Oct 23, 2024

Conversation

aptkingston
Copy link
Member

@aptkingston aptkingston commented Sep 13, 2024

Description

image

This PR adds a new visual way of defining roles for your apps. Custom roles can be created, edited and deleted using this new RBAC editor, and inheritance can be defined using intuitive drag-and-drop connections. Inheriting multiple roles is now possible too.

Clarification of functionality:

  • Power user role remains for existing apps, but will not be created in new apps
  • Power user role cannot be edited and cannot inherit custom roles, but custom roles may inherit it
  • Fully backwards compatible with existing custom roles structures
  • Full multiplayer support for custom role CRUD
  • Positions of roles are not currently saved, by design. The layout will reset to auto-layout when refreshing the page.

Feature branch env

Feature Branch Link

aptkingston and others added 30 commits September 9, 2024 17:04
…alidator not allowing the structure that roles are expected to have.
@aptkingston aptkingston requested a review from a team as a code owner October 18, 2024 10:56
@aptkingston aptkingston requested review from samwho and removed request for a team October 18, 2024 10:56
@@ -177,6 +178,7 @@ export async function save(ctx: UserCtx<SaveRoleRequest, SaveRoleResponse>) {
},
})
}
builderSocket?.emitRoleUpdate(ctx, role)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only backend change is multiplayer support for role changes.

Copy link
Contributor

@deanhannigan deanhannigan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Any issues found during testing are now resolved 🎉 . Fantastic work @aptkingston. The new role UX is amazing!

})
</script>

<div class="dev-preview-header" class:mobile={$context.device.mobile}>
<Heading size="XS">Preview</Heading>
<Heading size="XS">Previ werewr eww</Heading>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥚

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you 😂 May have been testing things...

@aptkingston aptkingston merged commit 45dbfd2 into v3-ui Oct 23, 2024
9 of 11 checks passed
@aptkingston aptkingston deleted the new-rbac-ui branch October 23, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-branch-enterprise Release this PR code into a feature branch with a Enterprise license size/xl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants