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

Add more fields to the standard role editor #50202

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

bl-nero
Copy link
Contributor

@bl-nero bl-nero commented Dec 13, 2024

Also relax some rules about the resource kind names in access rules. All of this allows us to support the predefined auditor and editor roles with a standard role editor.

Screenshot 2024-12-11 at 17 20 56

Screenshot 2024-12-11 at 16 49 01

Screenshot 2024-12-11 at 16 51 23

Contributes to #46612

Also relax some rules about the resource kind names in access rules.
@bl-nero bl-nero added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v17 labels Dec 13, 2024
@bl-nero bl-nero requested review from avatus and strideynet December 13, 2024 12:01
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-50202.d3pp5qlev8mo18.amplifyapp.com

@bl-nero bl-nero added this pull request to the merge queue Dec 13, 2024
Merged via the queue into master with commit 63ef960 Dec 13, 2024
43 of 44 checks passed
@bl-nero bl-nero deleted the bl-nero/role-editor-more-fields branch December 13, 2024 12:50
@public-teleport-github-review-bot

@bl-nero See the table below for backport results.

Branch Result
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry size/md ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants