Skip to content

[7.x] Trigger context (#57870)#58224

Merged
streamich merged 1 commit intoelastic:7.xfrom
streamich:backport/7.x/pr-57870
Feb 21, 2020
Merged

[7.x] Trigger context (#57870)#58224
streamich merged 1 commit intoelastic:7.xfrom
streamich:backport/7.x/pr-57870

Conversation

@streamich
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* feat: 🎸 annotate with comments Trigger and add createContext(

* feat: 🎸 add TriggerContext<T> type

* feat: 🎸 improve trigger setup in embeddable plugin

* feat: 🎸 export trigger vars and types from embeddable

* feat: 🎸 add internal representation of Trigger

* feat: 🎸 use new trigger interface to execute in vis embeddable

* feat: 🎸 add TriggerContextMapping interface

* feat: 🎸 improve trigger types

* refactor: 💡 remove trigger ID getter

* chore: 🤖 remove commented out line

* chore: 🤖 re-export more trigger stuff

* refactor: 💡 simplify Trigger interface

* suggestions

* Remove unused type

Co-authored-by: Stacey Gammon <gammon@elastic.co>
@streamich streamich added the backport This PR is a backport of another PR label Feb 21, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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

@streamich streamich merged commit c771007 into elastic:7.x Feb 21, 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