[pull] main from SigNoz:main#746
Merged
Merged
Conversation
* fix: skip resource filter if it exceeds a threshold * fix: test cleanup * fix: test cleanup * fix: more cleanup * fix: tests
…es after component migration (#11546) * fix: resolve UX regressions across dashboards, metrics and alerts pages after component migration - NewSelect: make the @signozhq checkbox label fill the row (min-width: 0) so long option text truncates with an ellipsis; size Only/Toggle buttons to the resting row height so hover no longer grows the row - PlannedDowntime: reset field-label styles bleeding into the migrated RadioGroup option labels and add spacing between options - Page-level contrast, spacing, alignment and icon-size fixes across dashboard settings, metrics explorer, alert rules, routing policies, auto-refresh and the time picker * fix(multiselect): surface clear icon and remove dropdown double-scroll - add .ant-select-clear color override so the multiselect clear (x) icon is visible on the dark selector (the single-select had it, multiselect didn't) - raise the dropdown max-height so the react-virtuoso list (<=300px) plus the header/footer fits, leaving the list as the single scroller instead of the list and the container both scrolling - StatsCard: drop the unnecessary !important on .count-label color * fix: success colors
Co-authored-by: primus-bot[bot] <171087277+primus-bot[bot]@users.noreply.github.com>
Replaces the `SuccessResponse<...> & { warning?: Warning; meta?: ExecStats }`
intersection with a single interface that extends SuccessResponse and declares
warning + meta on its own — no stitched intersection. Also types `params` as
`QueryRangeRequestV5` (was `unknown`), so callers can drop the redundant
`as QueryRangeRequestV5` casts.
Knock-on updates: 13 consumers switched from the raw
`SuccessResponse<MetricRangePayloadProps, unknown>` shape to the new
`MetricQueryRangeSuccessResponse`; 3 prod casts removed;
convertV5ResponseToLegacy and getTimeRange tightened their accept-params types.
* fix: show warning for non-existent cost meter metrics * chore: lint fix by removing unused list * chore: py fmt add new line * chore: missing newline between tests * fix: no warnings or errors for internal metrics * fix: pylint fix by adding new line * fix: lint fix in test
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )