-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[File data visualizer] Disabling create data view based on capabilities #117347
[File data visualizer] Disabling create data view based on capabilities #117347
Conversation
Pinging @elastic/ml-ui (:ml) |
...ata_visualizer/public/application/file_data_visualizer/components/import_view/import_view.js
Outdated
Show resolved
Hide resolved
showTooltip ? ( | ||
<FormattedMessage | ||
id="xpack.dataVisualizer.file.cannotCreateDataView.tooltip" | ||
defaultMessage="You require permission to create data views" |
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.
Minor suggestion
defaultMessage="You require permission to create data views" | |
defaultMessage="You need permission to create data views" |
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.
Updated in 93f540c
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.
Tested and LGTM
💚 Build Succeeded
Metrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
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.
Code LGTM
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
As discussed, would be good to apply this same check in the data frame analytics and transform wizards, to disable the 'Create data view' controls there too, but can be done in separate PR(s).
…es (elastic#117347) * [File data visualizer] Disabling create data view based on capabilities * removing data view management card based on permissions * fixing reset button and index data viz link
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
…es (#117347) (#117520) * [File data visualizer] Disabling create data view based on capabilities * removing data view management card based on permissions * fixing reset button and index data viz link Co-authored-by: James Gowdy <[email protected]>
If the user does not have permission to create index patterns, we should disable the option in the UI.
Also removes the link to manage data views if the user does not have permission.
Also removes link to the index data visualizer if the index pattern hasn't been created. (ML only)