Skip to content
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

Update KQL/filter bar suggestions to cancel when navigating away #69769

Merged
merged 3 commits into from
Jul 1, 2020

Conversation

lukasolson
Copy link
Member

@lukasolson lukasolson commented Jun 23, 2020

Summary

Closes #48797.

Updates the KQL & filter bar to properly cancel value suggestions requests when the input changes or the user navigates away.

Prior to this PR, we were properly cancelling KQL value suggestion requests when the query changed, but not when users navigate away or in filter bar suggestions. This PR adds the remainder of the cancellation functionality.

Release notes

When using KQL or the filter bar, if a request is issued to Elasticsearch for suggestions for a value for a specific field, the request will be cancelled when navigating away as well as when the input is updated (and a new request is issued).

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-arch (Team:AppArch)

@alexwizp
Copy link
Contributor

@elasticmachine merge upstream

@alexwizp alexwizp added release_note:skip Skip the PR/issue when compiling release notes release_note:fix and removed release_note:skip Skip the PR/issue when compiling release notes labels Jun 25, 2020
@lizozom
Copy link
Contributor

lizozom commented Jun 30, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

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

Copy link
Contributor

@lizozom lizozom left a comment

Choose a reason for hiding this comment

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

Tested locally. LGTM.

@lukasolson lukasolson merged commit ee621d0 into elastic:master Jul 1, 2020
lukasolson added a commit to lukasolson/kibana that referenced this pull request Jul 1, 2020
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 3, 2020
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

1 similar comment
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 7, 2020
lukasolson added a commit that referenced this pull request Jul 7, 2020
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.

[KQL] [Filter editor] Cancel discarded requests for suggestions
5 participants