Skip to content

feat(dashboards): adds validateOnly param to dry run and short circuit dashboard save#110746

Merged
edwardgou-sentry merged 1 commit intomasterfrom
edwardgou/browse-442-create-dashboard-dry-run-param
Mar 16, 2026
Merged

feat(dashboards): adds validateOnly param to dry run and short circuit dashboard save#110746
edwardgou-sentry merged 1 commit intomasterfrom
edwardgou/browse-442-create-dashboard-dry-run-param

Conversation

@edwardgou-sentry
Copy link
Contributor

@edwardgou-sentry edwardgou-sentry commented Mar 16, 2026

Adds a validateOnly param to run the dashboards saving endpoint without actually committing to the database. This is for the purpose of checking dashboards payloads for validation.

@linear-code
Copy link

linear-code bot commented Mar 16, 2026

@edwardgou-sentry edwardgou-sentry marked this pull request as ready for review March 16, 2026 16:31
@edwardgou-sentry edwardgou-sentry requested a review from a team as a code owner March 16, 2026 16:31
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 16, 2026
Copy link
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.

It might be good if we emitted some metrics/span with some meta data to track instances of failures and reasons

@edwardgou-sentry edwardgou-sentry merged commit 3e0d070 into master Mar 16, 2026
61 checks passed
@edwardgou-sentry edwardgou-sentry deleted the edwardgou/browse-442-create-dashboard-dry-run-param branch March 16, 2026 19:28
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