Skip to content

feat(dashboards): Track metrics for Seer dashboard create and edit flows#112595

Merged
edwardgou-sentry merged 4 commits intomasterfrom
egou/feat/track-gen-dashboard-edit-flow-metrics
Apr 9, 2026
Merged

feat(dashboards): Track metrics for Seer dashboard create and edit flows#112595
edwardgou-sentry merged 4 commits intomasterfrom
egou/feat/track-gen-dashboard-edit-flow-metrics

Conversation

@edwardgou-sentry
Copy link
Copy Markdown
Contributor

Summary

  • Adds dashboards.seer.create.save metric when a user saves a Seer-generated dashboard for the first time
  • Adds dashboards.seer.edit.save metric when a user saves AI-suggested changes on an existing dashboard
  • Adds source attribute (create | edit) to dashboards.seer.validation metric to distinguish between flows
  • Refactors validateDashboardAndRecordMetrics into shared createFromSeerUtils and threads seerRunId through the edit flow callbacks

@edwardgou-sentry edwardgou-sentry requested a review from a team as a code owner April 9, 2026 17:28
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 9, 2026
Comment thread static/app/views/dashboards/detail.tsx Outdated
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 47f00f4. Configure here.

Comment thread static/app/views/dashboards/detail.tsx
Comment thread static/app/views/dashboards/detail.tsx
The validateDashboardAndRecordMetrics call was inside the setState
updater function, which React Strict Mode can double-invoke. Move it
to the setState callback to ensure it runs exactly once after state
is committed.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Convert seer_run_id to number in create save metric to match the
edit flow. Reset seerEditApplied and seerRunId in the isEqual early
return path to prevent stale state from triggering false edit metrics.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@edwardgou-sentry edwardgou-sentry merged commit c107292 into master Apr 9, 2026
73 of 75 checks passed
@edwardgou-sentry edwardgou-sentry deleted the egou/feat/track-gen-dashboard-edit-flow-metrics branch April 9, 2026 19:43
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
…ows (#112595)

- Adds `dashboards.seer.create.save` metric when a user saves a
Seer-generated dashboard for the first time
- Adds `dashboards.seer.edit.save` metric when a user saves AI-suggested
changes on an existing dashboard
- Adds `source` attribute (`create` | `edit`) to
`dashboards.seer.validation` metric to distinguish between flows
- Refactors `validateDashboardAndRecordMetrics` into shared
`createFromSeerUtils` and threads `seerRunId` through the edit flow
callbacks

---------

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

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