Skip to content

[ML] Adding missing detector descriptions to advanced wizard#48440

Merged
jgowdyelastic merged 3 commits intoelastic:masterfrom
jgowdyelastic:adding-detector-descriptions-to-advanced-wizard
Oct 18, 2019
Merged

[ML] Adding missing detector descriptions to advanced wizard#48440
jgowdyelastic merged 3 commits intoelastic:masterfrom
jgowdyelastic:adding-detector-descriptions-to-advanced-wizard

Conversation

@jgowdyelastic
Copy link
Copy Markdown
Member

@jgowdyelastic jgowdyelastic commented Oct 16, 2019

Adds the missing detector descriptions to the advanced wizard.

2019-10-16 20-07-41 2019-10-16 20_10_52

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

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@jgowdyelastic jgowdyelastic changed the title [ML] Adding detector descriptions to advanced wizard [ML] Adding missing detector descriptions to advanced wizard Oct 16, 2019
@jgowdyelastic jgowdyelastic self-assigned this Oct 16, 2019
@jgowdyelastic jgowdyelastic added :ml non-issue Indicates to automation that a pull request should not appear in the release notes release_note:skip Skip the PR/issue when compiling release notes review v7.5.0 v7.6.0 v8.0.0 labels Oct 16, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@jgowdyelastic jgowdyelastic marked this pull request as ready for review October 16, 2019 19:15
@jgowdyelastic jgowdyelastic requested a review from a team as a code owner October 16, 2019 19:15
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Copy link
Copy Markdown
Contributor

@alvarezmelissa87 alvarezmelissa87 left a comment

Choose a reason for hiding this comment

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

LGTM ⚡️

Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

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

LGTM

@jgowdyelastic jgowdyelastic merged commit 8cb331f into elastic:master Oct 18, 2019
@jgowdyelastic jgowdyelastic deleted the adding-detector-descriptions-to-advanced-wizard branch October 18, 2019 08:12
jgowdyelastic added a commit to jgowdyelastic/kibana that referenced this pull request Oct 18, 2019
…#48440)

* [ML] Adding detector descriptions to advanced wizard

* renaming things

* reverting accidentally added refactor
jgowdyelastic added a commit to jgowdyelastic/kibana that referenced this pull request Oct 18, 2019
…#48440)

* [ML] Adding detector descriptions to advanced wizard

* renaming things

* reverting accidentally added refactor
jgowdyelastic added a commit that referenced this pull request Oct 18, 2019
…#48613)

* [ML] Adding detector descriptions to advanced wizard

* renaming things

* reverting accidentally added refactor
jgowdyelastic added a commit that referenced this pull request Oct 18, 2019
…#48615)

* [ML] Adding detector descriptions to advanced wizard

* renaming things

* reverting accidentally added refactor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:ml non-issue Indicates to automation that a pull request should not appear in the release notes release_note:skip Skip the PR/issue when compiling release notes review v7.5.0 v7.6.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants