Skip to content

[ML] Fix FileViz new ML Job link shown without canCreateJob permission - #27612

Merged
alvarezmelissa87 merged 1 commit into
elastic:masterfrom
alvarezmelissa87:ml-fileviz-new-job-link
Dec 21, 2018
Merged

alvarezmelissa87 merged 1 commit into
elastic:masterfrom
alvarezmelissa87:ml-fileviz-new-job-link

Conversation

@alvarezmelissa87

Copy link
Copy Markdown
Contributor

Summary

Fix for: #25804

Use same permission check for showing create new ML job after file import in fileViz as Create New Job button in JobManagement > JobsList.

  • Only shows link if checkPermission('canCreateJob') && mlNodesAvailable()

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
- [ ] Documentation was added for features that require explanation or tutorials
- [ ] Unit or functional tests were updated or added to match the most common scenarios
- [ ] This was checked for keyboard-only and screenreader accessibility

For maintainers

- [ ] This was checked for breaking API changes and was labeled appropriately
- [ ] This includes a feature addition or change that requires a release note and was labeled appropriately

@alvarezmelissa87 alvarezmelissa87 added bug Fixes for quality problems that affect the customer experience review v7.0.0 :ml Feature:Anomaly Detection ML anomaly detection v6.6.0 v6.7.0 labels Dec 20, 2018
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui

@jgowdyelastic jgowdyelastic left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@peteharverson peteharverson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@walterra walterra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@peteharverson peteharverson changed the title [ML] Fix FileViz newMLJob link shown without canCreateJob permission [ML] Fix FileViz new ML Job link shown without canCreateJob permission Dec 21, 2018
@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience Feature:Anomaly Detection ML anomaly detection :ml review v6.6.0 v6.7.0 v7.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants