Skip to content

[7.x] [Alerting] Adds generic UI for the definition of conditions for Action Groups (#83278)#83896

Merged
gmmorris merged 1 commit intoelastic:7.xfrom
gmmorris:backport/7.x/pr-83278
Nov 20, 2020
Merged

[7.x] [Alerting] Adds generic UI for the definition of conditions for Action Groups (#83278)#83896
gmmorris merged 1 commit intoelastic:7.xfrom
gmmorris:backport/7.x/pr-83278

Conversation

@gmmorris
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…n Groups (elastic#83278)

This PR adds two components to aid in creating a uniform UI for specifying the conditions for Action Groups:
1. `AlertConditions`: A component that generates a container which renders custom component for each Action Group which has had its _conditions_ specified.
2. `AlertConditionsGroup`: A component that provides a unified container for the Action Group with its name and a button for resetting its condition.

This can be used by any Alert Type to easily create the UI for adding action groups with whichever UI is specific to their component.
@gmmorris gmmorris added the backport This PR is a backport of another PR label Nov 20, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
triggersActionsUi 275 277 +2

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
triggersActionsUi 1.4MB 1.4MB +941.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
triggersActionsUi 143.3KB 152.3KB +9.1KB

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

@gmmorris gmmorris merged commit addf1a6 into elastic:7.x Nov 20, 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.

2 participants