Skip to content

fix(slo): Hide actions for read-only user#226284

Merged
kdelemme merged 1 commit intoelastic:mainfrom
kdelemme:slo/fix-management-actions
Jul 2, 2025
Merged

fix(slo): Hide actions for read-only user#226284
kdelemme merged 1 commit intoelastic:mainfrom
kdelemme:slo/fix-management-actions

Conversation

@kdelemme
Copy link
Contributor

@kdelemme kdelemme commented Jul 2, 2025

Summary

Resolves #226258

This PR hides the bulk menu and disable the clone action for readonly users on the SLO management page.

Testing

  1. Create some SLOs
  2. Create a read only user, logged in
  3. Go to the SLO management page
  4. Assert every actions are disabled
  5. Assert bulk actions are not available

@kdelemme kdelemme self-assigned this Jul 2, 2025
@github-actions github-actions bot added the author:obs-ux-management PRs authored by the obs ux management team label Jul 2, 2025
@kdelemme kdelemme added release_note:fix Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. backport:version Backport to applied version labels v9.1.0 v8.19.0 v9.2.0 release_note:skip Skip the PR/issue when compiling release notes and removed author:obs-ux-management PRs authored by the obs ux management team release_note:fix labels Jul 2, 2025
@kdelemme kdelemme marked this pull request as ready for review July 2, 2025 18:14
@kdelemme kdelemme requested a review from a team as a code owner July 2, 2025 18:14
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@kdelemme kdelemme enabled auto-merge (squash) July 2, 2025 18:36
@kdelemme kdelemme merged commit 7e286e0 into elastic:main Jul 2, 2025
28 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.19, 9.1

https://github.com/elastic/kibana/actions/runs/16034371341

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
slo 988.2KB 988.2KB +87.0B

cc @kdelemme

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 2, 2025
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 2, 2025
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.19
9.1

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jul 2, 2025
# Backport

This will backport the following commits from `main` to `9.1`:
- [fix(slo): Hide actions for read-only user
(#226284)](#226284)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Kevin
Delemme","email":"kevin.delemme@elastic.co"},"sourceCommit":{"committedDate":"2025-07-02T19:47:32Z","message":"fix(slo):
Hide actions for read-only user
(#226284)","sha":"7e286e085c777847da673f711e862ea685a7f61a","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:obs-ux-management","backport:version","v9.1.0","v8.19.0","v9.2.0"],"title":"fix(slo):
Hide actions for read-only
user","number":226284,"url":"https://github.com/elastic/kibana/pull/226284","mergeCommit":{"message":"fix(slo):
Hide actions for read-only user
(#226284)","sha":"7e286e085c777847da673f711e862ea685a7f61a"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/226284","number":226284,"mergeCommit":{"message":"fix(slo):
Hide actions for read-only user
(#226284)","sha":"7e286e085c777847da673f711e862ea685a7f61a"}}]}]
BACKPORT-->

Co-authored-by: Kevin Delemme <kevin.delemme@elastic.co>
kibanamachine added a commit that referenced this pull request Jul 2, 2025
# Backport

This will backport the following commits from `main` to `8.19`:
- [fix(slo): Hide actions for read-only user
(#226284)](#226284)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Kevin
Delemme","email":"kevin.delemme@elastic.co"},"sourceCommit":{"committedDate":"2025-07-02T19:47:32Z","message":"fix(slo):
Hide actions for read-only user
(#226284)","sha":"7e286e085c777847da673f711e862ea685a7f61a","branchLabelMapping":{"^v9.2.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:obs-ux-management","backport:version","v9.1.0","v8.19.0","v9.2.0"],"title":"fix(slo):
Hide actions for read-only
user","number":226284,"url":"https://github.com/elastic/kibana/pull/226284","mergeCommit":{"message":"fix(slo):
Hide actions for read-only user
(#226284)","sha":"7e286e085c777847da673f711e862ea685a7f61a"}},"sourceBranch":"main","suggestedTargetBranches":["9.1","8.19"],"targetPullRequestStates":[{"branch":"9.1","label":"v9.1.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.2.0","branchLabelMappingKey":"^v9.2.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/226284","number":226284,"mergeCommit":{"message":"fix(slo):
Hide actions for read-only user
(#226284)","sha":"7e286e085c777847da673f711e862ea685a7f61a"}}]}]
BACKPORT-->

Co-authored-by: Kevin Delemme <kevin.delemme@elastic.co>
kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. v8.19.0 v9.1.0 v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SLO] Fix available actions for read only users on management page

4 participants