Skip to content

fix(dashboards): Allow table widgets to have a limit up to 20#110621

Merged
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1333-allow-table-widget-limit-20
Mar 13, 2026
Merged

fix(dashboards): Allow table widgets to have a limit up to 20#110621
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1333-allow-table-widget-limit-20

Conversation

@DominikB2014
Copy link
Contributor

The backend widget serializer validates widget limits by display type, but had no
specific case for table widgets — they fell through to the default max of 10. The
frontend hardcodes a table limit of 20 (via TABLE_ITEM_LIMIT in sortableWidget.tsx),
so saving or updating table widgets with the default limit would fail validation.

Adds TABLE as a recognized display type with max_allowed = 20 in the validation chain.

The backend validator was rejecting table widget limits above 10,
but table widgets on the frontend use a limit of 20. This adds a
specific max_allowed of 20 for table display types.
@DominikB2014 DominikB2014 marked this pull request as ready for review March 13, 2026 13:10
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 13, 2026
@DominikB2014 DominikB2014 requested a review from a team as a code owner March 13, 2026 13:10
@getsentry getsentry deleted a comment from linear-code bot Mar 13, 2026
@DominikB2014 DominikB2014 enabled auto-merge (squash) March 13, 2026 13:10
@DominikB2014 DominikB2014 merged commit 7ed7c92 into master Mar 13, 2026
59 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1333-allow-table-widget-limit-20 branch March 13, 2026 13:31
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.

3 participants