Skip to content

[8.11] Fix autocomplete triggering on URL tokens (#168956)#169597

Merged
kibanamachine merged 1 commit intoelastic:8.11from
kibanamachine:backport/8.11/pr-168956
Oct 24, 2023
Merged

[8.11] Fix autocomplete triggering on URL tokens (#168956)#169597
kibanamachine merged 1 commit intoelastic:8.11from
kibanamachine:backport/8.11/pr-168956

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.11:

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Oct 23, 2023
@kibanamachine kibanamachine enabled auto-merge (squash) October 23, 2023 22:51
## Summary

This PR fixes autocomplete triggering on URL tokens ~~for url parameters
to work with a single character~~.

Fixes elastic#168017 (which is a regression introduced by elastic#163233)

![fix-autocomplete-168017](https://github.com/elastic/kibana/assets/721858/94e6f773-53c9-4bc1-991c-fb572ddf7ffd)

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

### Notes

- No functional tests are added because they would also go flaky.
- ~~No unit tests are added because of the lack of existing unit
tests.~~
- ~~The change is kept minimal by accepting the growing if-else block.~~

### For maintainers

- [x] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

---------

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 8fe2e1a)
@kibanamachine kibanamachine force-pushed the backport/8.11/pr-168956 branch from 2242249 to 8e6e91c Compare October 23, 2023 22:59
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
console 199 200 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
console 403.6KB 404.6KB +1.0KB

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

cc @sakurai-youhei

@kibanamachine kibanamachine merged commit 2fa72af into elastic:8.11 Oct 24, 2023
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.

3 participants