Skip to content

[9.1] [dashboard] fix unable to reset unsaved change when enabling timeRestore and setting time range (#239992)#240111

Closed
kibanamachine wants to merge 1 commit intoelastic:9.1from
kibanamachine:backport/9.1/pr-239992
Closed

[9.1] [dashboard] fix unable to reset unsaved change when enabling timeRestore and setting time range (#239992)#240111
kibanamachine wants to merge 1 commit intoelastic:9.1from
kibanamachine:backport/9.1/pr-239992

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.1:

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

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

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

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

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

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

Questions ?

Please refer to the Backport tool documentation

…ore and setting time range (elastic#239992)

Closes elastic#239962

The issue was caused because `timeRestore$` is used in comparators but
was not part of `startComparing$` subscription. The unsaved changes
badge would remain because `startComparing$` from unifiedSearchManager
would not emit after `timeRestore$` was reset.

Issue resolved by adding `timeRestore$` to `startComparing$`
subscription. Now `timeRestore$` emit will cause `startComparing$` to
emit with freshly calculated values.

(cherry picked from commit 6a78829)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Oct 22, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) October 22, 2025 14:23
@elasticmachine
Copy link
Contributor

elasticmachine commented Oct 22, 2025

💔 Build Failed

Failed CI Steps

History

cc @nreese

@nreese
Copy link
Contributor

nreese commented Oct 22, 2025

Closing, replacing with #240143

@nreese nreese closed this Oct 22, 2025
auto-merge was automatically disabled October 22, 2025 17:01

Pull request was closed

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