-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Vega] Add Filter custom label for kibanaAddFilter #124498
Conversation
Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually? |
Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors) |
@elasticmachine merge upstream |
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.
LGTM! Tested locally
Buildkite, test this |
@elasticmachine merge upstream |
Buildkite, test this |
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.
Tested and LGTM, thanks for your contribution! I added the feature to the documentation
@elasticmachine run elasticsearch-ci/docs |
💚 Build SucceededMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: cc @alexwizp |
Summary
Vega 'kibanaAddFilter' allows to add Elastic Query DSL and index. The implementation buildQueryFilter takes in third parameter to set a custom label in 'Add Filter'. At the moment, kibanaAddFilter can only take query and index. This PR adds in the third 'alias' parameter to set the custom label from Vega
buildQueryFilter = (query: QueryStringFilter['query'], index: string, alias: string)
Below image is self explanatory. The first filter in the image is without custom label (ugly in my opinion), and the second filter is with custom label set in Vega.