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 #50067

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

bl-nero
Copy link
Contributor

@bl-nero bl-nero commented Dec 11, 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 11, 2024
@github-actions github-actions bot requested review from avatus and kiosion December 11, 2024 16:23
@bl-nero bl-nero force-pushed the bl-nero/role-editor-more-fields branch from 49b40c2 to ae5acea Compare December 11, 2024 17:38
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from kiosion December 13, 2024 11:48
@bl-nero bl-nero merged commit ea1ee3d into bl-nero/role-editor-base Dec 13, 2024
79 checks passed
@bl-nero bl-nero deleted the bl-nero/role-editor-more-fields branch December 13, 2024 11:49
bl-nero added a commit that referenced this pull request Dec 13, 2024
@public-teleport-github-review-bot

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

Branch Result
branch/v17 Create PR

@bl-nero bl-nero restored the bl-nero/role-editor-more-fields branch December 13, 2024 11:50
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