Skip to content

[7.x] Make the alerts plugin support generics (#72716)#75078

Merged
mikecote merged 2 commits intoelastic:7.xfrom
mikecote:backport/7.x/pr-72716
Aug 17, 2020
Merged

[7.x] Make the alerts plugin support generics (#72716)#75078
mikecote merged 2 commits intoelastic:7.xfrom
mikecote:backport/7.x/pr-72716

Conversation

@mikecote
Copy link
Contributor

Backports the following commits to 7.x:

* Initial work

* Expand generic support to alert instances

* Convert index threshold to use generics

* Make fixture alert types use generics

* Make alert instance related types use unknown

* Fix typecheck failures

* Cleanup + add instance generic support to registry.get API

* Shallow clone

* Rename some TS variables

* Fix failing api integration tests

* Change code for easier review and keep more history

* Fix

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@mikecote mikecote added the backport This PR is a backport of another PR label Aug 14, 2020
@mikecote
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

page load bundle size

id value diff baseline
alerts 89.3KB +155.0B 89.1KB

History

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

@mikecote mikecote merged commit 1433c88 into elastic:7.x Aug 17, 2020
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.

3 participants