Skip to content

[8.19] [Lens] Keep suggestions up to date with the main workspace (#221901)#222380

Merged
dej611 merged 1 commit intoelastic:8.19from
dej611:backport/8.19/pr-221901
Jun 3, 2025
Merged

[8.19] [Lens] Keep suggestions up to date with the main workspace (#221901)#222380
dej611 merged 1 commit intoelastic:8.19from
dej611:backport/8.19/pr-221901

Conversation

@dej611
Copy link
Contributor

@dej611 dej611 commented Jun 3, 2025

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…21901)

## Summary

Fixes elastic#178571

This PR reverts elastic#221248 adding the missing part on it to make the bug
found in elastic#178571 disappear.
In fact there was just a single line missing who was meant to keep the
suggestion frame version up to date with the latest workspace state.

![lens_suggestion_bug_fix](https://github.com/user-attachments/assets/ff59e4d6-cfac-4801-857a-a79b1c98762d)

I've tried to add a unit test but it was so hard to actually follow the
code underneath, so I settled to add a FTR, which led to few more
extras:

* subtype selector now has test ids
* Lens page now has few more utility helpers for layer cloning and sub
type selector

### 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

(cherry picked from commit 0bcd7ff)

# Conflicts:
#	x-pack/test/functional/page_objects/lens_page.ts
@dej611 dej611 requested a review from kibanamachine as a code owner June 3, 2025 15:26
@dej611 dej611 added the backport This PR is a backport of another PR label Jun 3, 2025
@dej611 dej611 enabled auto-merge (squash) June 3, 2025 15:26
@dej611 dej611 merged commit b798a4e into elastic:8.19 Jun 3, 2025
11 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
lens 1.5MB 1.5MB +580.0B

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