Skip to content

[8.16] [Controls] Persist `runPastTimeout` setting (#208611)#208763

Merged
kibanamachine merged 1 commit intoelastic:8.16from
kibanamachine:backport/8.16/pr-208611
Jan 29, 2025
Merged

[8.16] [Controls] Persist `runPastTimeout` setting (#208611)#208763
kibanamachine merged 1 commit intoelastic:8.16from
kibanamachine:backport/8.16/pr-208611

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.16:

Questions ?

Please refer to the Backport tool documentation

Questions ?

Please refer to the Backport tool documentation

Questions ?

Please refer to the Backport tool documentation

Questions ?

Please refer to the Backport tool documentation

Closes elastic#206459

## Summary

This PR ensures that the `runPastTimeout` setting is persisted for
options list controls and the UI reflects the value of this setting. The
root cause of this bug is that we weren't sending in the behavior
subject for `runPastTimeout$` to the data control's `editorStateManager`
when initializing the options list control, which meant that its value
could not be set by the options list's `CustomOptionsComponent` - this
PR fixes that.

**Before**

https://github.com/user-attachments/assets/2c9eeab8-67d9-46bc-938e-4d7cb91e435f

**After**

https://github.com/user-attachments/assets/d06b6ffa-b1e9-4ecd-b732-69bd69a8aee9

### 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
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit da41b8b)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Jan 29, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) January 29, 2025 14:52
@kibanamachine kibanamachine merged commit 15981af into elastic:8.16 Jan 29, 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
controls 458.6KB 458.6KB +17.0B

cc @Heenawter

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

3 participants