-
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
[ML] Fixes permissions checks for data visualizer create job links #55431
[ML] Fixes permissions checks for data visualizer create job links #55431
Conversation
Pinging @elastic/ml-ui (:ml) |
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
@@ -131,7 +133,10 @@ export const Page: FC = () => { | |||
const defaults = getDefaultPageState(); | |||
|
|||
const [showActionsPanel] = useState( |
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.
I think we can remove useState
here since the value gets never updated and just do const showActionsPanel = ...
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.
Done in 901a166
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.
Latest changes LGTM
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
…lastic#55431) * [ML] Fixes permissions checks for data visualizer create job links * [ML] Edits to permissions check following review * [ML] Revert unintentional additions
…lastic#55431) * [ML] Fixes permissions checks for data visualizer create job links * [ML] Edits to permissions check following review * [ML] Revert unintentional additions
…t-of-legacy * 'master' of github.com:elastic/kibana: [Watcher] Add support for additional watch action statuses (elastic#55092) [ML] Fix entity controls update. (elastic#55524) [ML] Fixing ML's watcher integration (elastic#55439) [ML] Fixes permissions checks for data visualizer create job links (elastic#55431) [SearchProfiler] Move out of legacy (elastic#55331) # Conflicts: # x-pack/plugins/watcher/server/models/action_status/action_status.js
Summary
Fixes the check used to determine whether or not to show the 'Create job' links in the index-based data visualizer.
As well as checking for a full license and a time-based index, checks are now also made that the user has the
canCreateJob
privilege and there are ML nodes available. If any of these checks return false, the side panel containing the links to the advanced wizard and any modules is hidden:For maintainers
Fixes #55178