Skip to content

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

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

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

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.2:

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)
@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 #240142

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

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