Skip to content

feat(dashboards): Adds Create Dashboard with Seer frontend modal and preview page#110451

Merged
edwardgou-sentry merged 4 commits intomasterfrom
egou/feat/generate-dashboard-frontend
Mar 11, 2026
Merged

feat(dashboards): Adds Create Dashboard with Seer frontend modal and preview page#110451
edwardgou-sentry merged 4 commits intomasterfrom
egou/feat/generate-dashboard-frontend

Conversation

@edwardgou-sentry
Copy link
Contributor

Adds a new Create Dashboard with Seer button:

  • Opens a GenerateDashboardFromSeerModal modal which accepts a prompt text field from the user
    • Clicking generate starts a seer run via the orgslug/dashboards/generate/ endpoint and navigates user to the CreateFromSeer preview page with the run_id
  • CreateFromSeer preview page polls the seer run_id and renders a dashboard preview using the artifact response when complete
image

@edwardgou-sentry edwardgou-sentry requested a review from a team as a code owner March 11, 2026 18:11
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 11, 2026
p {
margin-bottom: 0;
}
`;
Copy link
Contributor

Choose a reason for hiding this comment

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

New styled component instead of core components

Low Severity

MessageBlock is a newly added styled() call wrapping MarkedText. Per the frontend guidelines, core components like Container (for padding, background, border-radius) and Text (for font-size, color) are preferred over new styled() calls. The styling here (padding, background, border-radius, font-size, color) maps directly to props available on those core layout and typography primitives.

Fix in Cursor Fix in Web

Triggered by project rule: Frontend guidelines

Copy link
Contributor

Choose a reason for hiding this comment

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

This one is valid, but not always possible

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.

code lgtm overall, there's a couple comments regarding flag checks which look like they could be valid but i'll leave it up to you to go through those

p {
margin-bottom: 0;
}
`;
Copy link
Contributor

Choose a reason for hiding this comment

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

This one is valid, but not always possible

Copy link
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 1 potential issue.

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

@edwardgou-sentry edwardgou-sentry merged commit 6355405 into master Mar 11, 2026
63 checks passed
@edwardgou-sentry edwardgou-sentry deleted the egou/feat/generate-dashboard-frontend branch March 11, 2026 19:33
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