Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dynamic actions #58216
Dynamic actions #58216
Changes from 49 commits
2dada48
0cae149
171a481
6cfc9e7
8e6c65a
d14d876
ac04c80
179ff8a
0796903
cfce078
bd0eaf3
3700583
6e2e30e
4403691
3796961
9ca0267
6e3d10f
dad08ba
12f09dc
60d11b8
70061bb
34c0843
7cbd860
93e17cb
8209a59
8982ca0
358d72c
59ef699
399fa80
95d1424
3c55bd4
199d8d3
cf3669a
b8bacce
e63e92d
22f3188
079a7c8
0279111
a05490d
bed623c
3e81792
7732c84
55b8721
a25cf30
a045b9c
d0f67ec
7fbd1fb
5883734
d056277
2700d00
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now that this has been merged to master after my typescript PR, this is a duplicate type, there is a file
action_definition
with aActionDefinition
type as well. I think we should use that one, and like @Dosant suggested, tie Config shape with the factory which I think should go in x-pack.