Skip to content

fix(dashboards): Allow wheel widgets to save with null limit#114094

Merged
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1584-saving-wheel-widgets-can-fail-with-null-limit
Apr 28, 2026
Merged

fix(dashboards): Allow wheel widgets to save with null limit#114094
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1584-saving-wheel-widgets-can-fail-with-null-limit

Conversation

@DominikB2014
Copy link
Copy Markdown
Contributor

@DominikB2014 DominikB2014 commented Apr 27, 2026

Fixes a bug where the wheel widget failed to save due to "limit is required". Wheel widget don't require a limit, it just queries for aggregates with no group by, so having a limit isn't necessary as there will always be the same number of results.

Fixes DAIN-1584

Wheel widgets render a single aggregated row and don't use `limit`, but
they include grouped columns. The widget serializer required a non-null
`limit` for any chart widget with columns that wasn't a TABLE or
BIG_NUMBER, which caused saves to fail. Exempt the WHEEL display type
from that check.

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

linear-code Bot commented Apr 27, 2026

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 27, 2026
@DominikB2014 DominikB2014 marked this pull request as ready for review April 28, 2026 13:02
@DominikB2014 DominikB2014 requested a review from a team as a code owner April 28, 2026 13:02
@DominikB2014 DominikB2014 enabled auto-merge (squash) April 28, 2026 13:05
@DominikB2014 DominikB2014 merged commit 77d042a into master Apr 28, 2026
59 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1584-saving-wheel-widgets-can-fail-with-null-limit branch April 28, 2026 14:00
cleptric pushed a commit that referenced this pull request May 5, 2026
Fixes a bug where the wheel widget failed to save due to "limit is
required". Wheel widget don't require a limit, it just queries for
aggregates with no group by, so having a limit isn't necessary as there
will always be the same number of results.

Fixes DAIN-1584

Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants