Skip to content

feat(dashboards): Hide detail/summary prebuilt dashboards from list by default#110625

Merged
DominikB2014 merged 3 commits intomasterfrom
dominikbuszowiecki/dain-1337-hide-sentry-built-details-dashboards
Mar 13, 2026
Merged

feat(dashboards): Hide detail/summary prebuilt dashboards from list by default#110625
DominikB2014 merged 3 commits intomasterfrom
dominikbuszowiecki/dain-1337-hide-sentry-built-details-dashboards

Conversation

@DominikB2014
Copy link
Copy Markdown
Contributor

@DominikB2014 DominikB2014 commented Mar 13, 2026 •
edited
Loading

Adds a hidden field to the PrebuiltDashboard config so certain prebuilt dashboards
can be excluded from the dashboards list API response while still being synced to the
database (so they remain accessible via direct navigation).

Detail and summary dashboards are navigated to contextually from their parent pages
rather than browsed from the main dashboards list, so showing them clutters the list.
Hidden dashboards can still be included by passing ?filter=showHidden.

@DominikB2014 @claude
feat(dashboards): Hide detail/summary prebuilt dashboards from list b…
48e4a55 
…y default

Add a `hidden` field to PrebuiltDashboard config to exclude certain
dashboards from the API response while still syncing them to the DB.
Hidden dashboards can be included by passing `?filter=showHidden`.

Marks detail and summary dashboards (Query Details, Domain Details,
Web Vitals Page Summary, Mobile Vitals App Starts/Screen Loads/
Screen Rendering, AI Agent/MCP detail pages, Frontend Assets Summary,
Queue Summary) as hidden since they are navigated to contextually
rather than browsed from the dashboards list.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@linear-code
Copy link
Copy Markdown

linear-code bot commented Mar 13, 2026

DAIN-1337 Hide sentry built details dashboards

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 13, 2026
@DominikB2014
fix(dashboards): Unhide AI Agents and MCP detail dashboards
60cfcc6 
Remove hidden flag from AI Agents Model Details, AI Agents Tool
Details, MCP Tool Details, MCP Resource Details, and MCP Prompt
Details prebuilt dashboards.
@DominikB2014 DominikB2014 marked this pull request as ready for review March 13, 2026 13:52
@DominikB2014 DominikB2014 requested a review from a team as a code owner March 13, 2026 13:52
cursor[bot]
cursor bot reviewed Mar 13, 2026
View reviewed changes
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.

Fix All in Cursor

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

@DominikB2014
fix(dashboards): Update prebuilt dashboard tests for hidden filtering
592826f 
Existing tests assumed all synced prebuilt dashboards appear in the
API response. Update assertions to account for hidden dashboards
being excluded from responses by default.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 13, 2026

Backend Test Failures

Failures on 471fdb1 in this run:

tests/sentry/dashboards/endpoints/test_organization_dashboards.py::OrganizationDashboardsTest::test_endpoint_creates_prebuilt_dashboards_when_none_exist — log 
tests/sentry/dashboards/endpoints/test_organization_dashboards.py:2187: in test_endpoint_creates_prebuilt_dashboards_when_none_exist
    assert len(matching_response_data) == 1
E   assert 0 == 1
E    +  where 0 = len([])
tests/sentry/dashboards/endpoints/test_organization_dashboards.py::OrganizationDashboardsTest::test_get_with_only_prebuilt — log 
tests/sentry/dashboards/endpoints/test_organization_dashboards.py:2283: in test_get_with_only_prebuilt
    assert len(response.data) == prebuilt_dashboards_count
E   assert 2 == 3
E    +  where 2 = len([{'createdBy': None, 'dateCreated': datetime.datetime(2026, 3, 13, 14, 2, 17, 333916, tzinfo=datetime.timezone.utc), '...atetime.datetime(2026, 3, 13, 14, 2, 17, 334679, tzinfo=datetime.timezone.utc), 'environment': [], 'filters': {}, ...}])
E    +    where [{'createdBy': None, 'dateCreated': datetime.datetime(2026, 3, 13, 14, 2, 17, 333916, tzinfo=datetime.timezone.utc), '...atetime.datetime(2026, 3, 13, 14, 2, 17, 334679, tzinfo=datetime.timezone.utc), 'environment': [], 'filters': {}, ...}] = <Response status_code=200, "application/json">.data

@DominikB2014 DominikB2014 enabled auto-merge (squash) March 13, 2026 14:09
@DominikB2014 DominikB2014 merged commit a94d664 into master Mar 13, 2026
58 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1337-hide-sentry-built-details-dashboards branch March 13, 2026 14:24
@sentry-release-bot sentry-release-bot bot mentioned this pull request Mar 15, 2026
Closed
3 tasks
@github-actions github-actions bot locked and limited conversation to collaborators Mar 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@cursor cursor[bot] cursor[bot] left review comments

@bcoe bcoe bcoe approved these changes

Assignees

No one assigned

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DominikB2014 @bcoe