Skip to content

fix(dashboards): Fix duplicating prebuilt dashboard 400 within dashboard page#104406

Merged
edwardgou-sentry merged 1 commit intomasterfrom
egou/fix/duplicate-prebuilt-dashboard
Dec 4, 2025
Merged

fix(dashboards): Fix duplicating prebuilt dashboard 400 within dashboard page#104406
edwardgou-sentry merged 1 commit intomasterfrom
egou/fix/duplicate-prebuilt-dashboard

Conversation

@edwardgou-sentry
Copy link
Copy Markdown
Contributor

The create dashboard endpoint expects a non empty interval for each widget. This interval attribute actually does not get used anywhere, but I've added 5m to each prebuilt dashboard widget to satisfy the validation.

There is probably some refactor work to do to get rid of interval from the widget serializer if its not being used at all.

@edwardgou-sentry edwardgou-sentry requested a review from a team as a code owner December 4, 2025 19:10
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 4, 2025
@edwardgou-sentry edwardgou-sentry requested a review from a team December 4, 2025 19:12
Copy link
Copy Markdown
Contributor

@DominikB2014 DominikB2014 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, I wonder if we should be programmatically doing this to limit duplication

@edwardgou-sentry edwardgou-sentry merged commit dddfb53 into master Dec 4, 2025
50 checks passed
@edwardgou-sentry edwardgou-sentry deleted the egou/fix/duplicate-prebuilt-dashboard branch December 4, 2025 20:01
@github-actions github-actions bot locked and limited conversation to collaborators Dec 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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