Skip to content

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

Merged
nreese merged 2 commits intoelastic:9.2from
nreese:backport/9.2/pr-239992
Oct 23, 2025
Merged

[9.2] [dashboard] fix unable to reset unsaved change when enabling timeRestore and setting time range (#239992)#240142
nreese merged 2 commits intoelastic:9.2from
nreese:backport/9.2/pr-239992

Conversation

@nreese
Copy link
Contributor

@nreese nreese commented Oct 22, 2025

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)
@nreese nreese merged commit 2bd81f2 into elastic:9.2 Oct 23, 2025
12 checks passed
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

2 participants