-
Notifications
You must be signed in to change notification settings - Fork 149
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(rbac): add rbac dynamic plugin #937
feat(rbac): add rbac dynamic plugin #937
Conversation
🦋 Changeset detectedLatest commit: 0d6528c The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The image is available at: |
A few questions I have:
|
Agree with @yuvalgalanti on point 2- as per our design and earlier implementations i have seen- Administration is placed above Settings in the left navigation. |
for point 2: yeah Administration will be above settings and that would be handled with #932 cc @gashcrumb for point 1: would recommend log an issue as this PRs scope is to enable plugin into showcase and point can can be handled as at plugins level |
/cc @PatAKnight |
e049827
to
a682650
Compare
Signed-off-by: Yi Cai <[email protected]>
Signed-off-by: Yi Cai <[email protected]>
a682650
to
8a1529a
Compare
The image is available at: |
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.
I think the version needs to be bumped, otherwise it's looking good! I'll do a bit more testing with a local change just to verify that.
Signed-off-by: Yi Cai <[email protected]>
…bac-plugin-on-showcase
The image is available at: |
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.
have it working locally now too, looks great!
Signed-off-by: Yi Cai <[email protected]>
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
The image is available at: |
Was able to test it locally. Thanks @ciiay /lgtm |
/lgtm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: debsmita1, gashcrumb, invincibleJai The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description
By this change, the RBAC dynamic plugin is included in the backstage-showcase container image, disabled by default.
Which issue(s) does this PR fix
Requires
janus-idp/backstage-plugins#1133
PR acceptance criteria
Please make sure that the following steps are complete:
How to test changes / Special notes to the reviewer
backstage-plugins/plugins/rbac
directory, runyarn export-dynamic
to export RBAC plugin UI as a dynamic pluginbackstage-showcase/dynamic-plugins-root/
directory, run the below command to install RBAC plugin UIbackstage-showcase/app-config.local.yaml
file, add the below changesdynamic-plugins.default.yaml
file Line 503, change disabled value todisabled: false
to enable RBAC plugin UI.Administration
button shows on the left side menu.Screen recording of test:
rhidp_1050_test_rbac.mov