Skip to content

[7.x] Allow registered alert types to be non-editable (#65606)#66416

Merged
YulNaumenko merged 1 commit intoelastic:7.xfrom
YulNaumenko:backport/7.x/pr-65606
May 13, 2020
Merged

[7.x] Allow registered alert types to be non-editable (#65606)#66416
YulNaumenko merged 1 commit intoelastic:7.xfrom
YulNaumenko:backport/7.x/pr-65606

Conversation

@YulNaumenko
Copy link
Contributor

Backports the following commits to 7.x:

* Allow registered alert types to be non-editable

* Fixed isUiEditEnabled values

* Fixed due to comments

* fixed failing tests

* Enable alert type selection per alert consumer, only 'alerting' consumer can display other consumers alert types, but in case if it isEditable

* fixed tests

* Removed consumer property from the client side alert type registry and added server side property producer which purpose is to manage a feature logic

* fixed type check

* Fixed tests and type checks

* Removed error message for non registered plugins

* Fixed failing tests

* Fixed due to comments

* fixed test

* -

* revert logic for requiresAppContext

* Added close toast after saving alert
@YulNaumenko YulNaumenko added the backport This PR is a backport of another PR label May 13, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

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

@YulNaumenko YulNaumenko merged commit 437e01c into elastic:7.x May 13, 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