Skip to content

feat(dashboards): Separate prebuilt dashboards in starred sidebar#110303

Merged
DominikB2014 merged 4 commits intomasterfrom
dominikbuszowiecki/dain-1302-separate-prebuilt-dashboards-from-non-prebuilt-on-starred
Mar 10, 2026
Merged

feat(dashboards): Separate prebuilt dashboards in starred sidebar#110303
DominikB2014 merged 4 commits intomasterfrom
dominikbuszowiecki/dain-1302-separate-prebuilt-dashboards-from-non-prebuilt-on-starred

Conversation

@DominikB2014
Copy link
Contributor

@DominikB2014 DominikB2014 commented Mar 10, 2026

Summary

  • Splits the starred dashboards sidebar into two sections: custom dashboards ("Starred Dashboards") and Sentry-built dashboards ("Starred Sentry Built Dashboards")
image
  • Prebuilt dashboards are identified by having a prebuiltId and are shown in a separate section below custom starred dashboards
  • Custom dashboards continue to support the dashboards-starred-reordering feature flag

Test plan

  • Star a custom dashboard and verify it appears in "Starred Dashboards"
  • Star a Sentry-built dashboard and verify it appears in "Starred Sentry Built Dashboards"
  • Verify sections only appear when they have items
  • Verify the reordering feature flag still works for custom dashboards

… sidebar

Split the starred dashboards sidebar section into two: custom dashboards
appear in "Starred Dashboards" and Sentry-built dashboards (those with
a prebuiltId) appear in a new "Starred Sentry Built Dashboards" section
below.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@linear-code
Copy link

linear-code bot commented Mar 10, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 10, 2026
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Deduplicate the starred dashboard rendering logic between custom and
prebuilt sections. Also fix optional chaining on dashboard.projects
to prevent a TypeError when projects is unexpectedly undefined.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Wrap the two filter calls into a single useMemo to avoid recomputing
on every render.

Co-Authored-By: Claude Opus 4.6 <noreply@example.com>
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.

@DominikB2014 DominikB2014 merged commit 78182cc into master Mar 10, 2026
60 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1302-separate-prebuilt-dashboards-from-non-prebuilt-on-starred branch March 10, 2026 14:52
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