Skip to content

Fix Kuery autocomplete in APM#21249

Merged
lukasolson merged 1 commit intoelastic:masterfrom
lukasolson:fix/apmKuery
Jul 25, 2018
Merged

Fix Kuery autocomplete in APM#21249
lukasolson merged 1 commit intoelastic:masterfrom
lukasolson:fix/apmKuery

Conversation

@lukasolson
Copy link
Contributor

When #20747 was implemented, it was done at first in a hack, which meant it was loaded on every page (even in APM). However, when it was changed to its own UI export type, this meant we needed to load it in the APM plugin. That's exactly what this PR does, so that autocomplete once again works.

Copy link
Member

@sorenlouv sorenlouv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this so quickly 👍

Copy link
Contributor

@spalger spalger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@elasticmachine
Copy link
Contributor

💔 Build Failed

@lukasolson lukasolson merged commit 8a6c531 into elastic:master Jul 25, 2018
lukasolson added a commit to lukasolson/kibana that referenced this pull request Jul 25, 2018
lukasolson added a commit to lukasolson/kibana that referenced this pull request Jul 25, 2018
@lukasolson
Copy link
Contributor Author

lukasolson commented Jul 25, 2018

6.4 (6.4.0): 2eee0ce
6.x (6.5.0): 861375d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants