Skip to content

[Fix] add to clause to allow read only users the image reporting share menu#163855

Closed
rshen91 wants to merge 1 commit intoelastic:8.9from
rshen91:fix-for-sdh-4016
Closed

[Fix] add to clause to allow read only users the image reporting share menu#163855
rshen91 wants to merge 1 commit intoelastic:8.9from
rshen91:fix-for-sdh-4016

Conversation

@rshen91
Copy link
Contributor

@rshen91 rshen91 commented Aug 14, 2023

Summary

This PR #153429 inadvertently created a bug where users in Dashboard could not generate reports if they have custom roles that aren't all.

Checklist

Delete any items that are not applicable to this PR.

@kibana-ci
Copy link

kibana-ci commented Aug 14, 2023

💔 Build Failed

Failed CI Steps

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
reporting 42.7KB 42.8KB +105.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 13 15 +2
securitySolution 415 419 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 14 16 +2
securitySolution 496 500 +4
total +6

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@rshen91
Copy link
Contributor Author

rshen91 commented Aug 14, 2023

still an issue in main closing this

@rshen91 rshen91 closed this Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants