Skip to content

fix(dashboards): Preserve polarity when clearing threshold values#109872

Merged
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1275-clearing-dashboard-thresholds-flips-higherlower-is-better
Mar 4, 2026
Merged

fix(dashboards): Preserve polarity when clearing threshold values#109872
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1275-clearing-dashboard-thresholds-flips-higherlower-is-better

Conversation

@DominikB2014
Copy link
Contributor

When all threshold max values are cleared in the widget builder, the entire
thresholds config was set to null, which lost the preferredPolarity
setting. When the user re-entered threshold values, the polarity defaulted
back to "lower is better" (-), even if they had previously selected
"higher is better" (+).

Now when clearing all max values, the preferredPolarity is preserved in
the thresholds config so it persists across clearing and re-entering values.

Fixes DAIN-1275

When all threshold max values are cleared, the preferredPolarity was
lost because the entire thresholds config was set to null. This caused
the polarity to flip back to the default "lower is better" when the
user re-entered threshold values.

Fixes DAIN-1275
Co-Authored-By: Claude <noreply@anthropic.com>
@linear
Copy link

linear bot commented Mar 4, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 4, 2026
@DominikB2014 DominikB2014 marked this pull request as ready for review March 4, 2026 16:01
@DominikB2014 DominikB2014 requested a review from a team as a code owner March 4, 2026 16:01
@DominikB2014 DominikB2014 merged commit 69aff58 into master Mar 4, 2026
61 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1275-clearing-dashboard-thresholds-flips-higherlower-is-better branch March 4, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants