Skip to content

[7.x] [SIEM] Add license check to ML Rule form (#60691)#60928

Merged
rylnd merged 1 commit intoelastic:7.xfrom
rylnd:backport/7.x/pr-60691
Mar 23, 2020
Merged

[7.x] [SIEM] Add license check to ML Rule form (#60691)#60928
rylnd merged 1 commit intoelastic:7.xfrom
rylnd:backport/7.x/pr-60691

Conversation

@rylnd
Copy link
Contributor

@rylnd rylnd commented Mar 23, 2020

Backports the following commits to 7.x:

* Gate ML Rules behind a license check

If they don't have a Platinum or Trial license, then we disable the ML
Card and provide them a link to the subscriptions marketing page.

* Add aria-describedby for new ML input fields

* Add data-test-subj to new ML input fields

* Remove unused prop

This is already passed as isLoading

* Fix capitalization on translation id

* Declare defaulted props as optional

* Gray out entire ML card when ML Rules are disabled

If we're editing an existing rule, or if the user has an insufficient
license, we disable both the card and its selectability. This is more
visually striking, and a more obvious CTA.
@rylnd rylnd added the backport This PR is a backport of another PR label Mar 23, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@rylnd rylnd merged commit 3d14b3d into elastic:7.x Mar 23, 2020
@rylnd rylnd deleted the backport/7.x/pr-60691 branch March 23, 2020 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants