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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
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.

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
Copy Markdown
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
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 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
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2026
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