-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[SecuritySolutions] Update asset criticality upload page visibility and permissions #180771
Conversation
d7acbea
to
83680c5
Compare
/ci |
51747ad
to
d95d2f7
Compare
/ci |
1 similar comment
/ci |
f079568
to
eb2eb9e
Compare
/ci |
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-entity-analytics (Team:Entity Analytics) |
@@ -196,6 +196,7 @@ | |||
"@kbn/core-http-server-mocks", | |||
"@kbn/data-service", | |||
"@kbn/core-chrome-browser", | |||
"@kbn/shared-ux-chrome-navigation" |
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.
auto-generated 8e1f50e
8e1f50e
to
5c1664f
Compare
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.
Nice, just a small comment
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.
LGTM!
Thanks Pablo for doing this 💯
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: cc @machadoum |
Summary
Related to #179891
Display error message in case user doesn't have write access to asset criticality index
Display error message if the user navigates to the page when the user advanced (UI) setting is disabled
Remove links from menus and global search if advanced (UI) setting is disabled
Recoding of what happens when you don't refresh the page:
Screen.Recording.2024-04-15.at.14.35.16.small.mov
Expected behaviour:
Checklist
Delete any items that are not applicable to this PR.