Skip to content

ref(dashboards): Remove Dashboard Templates feature#113343

Open
gggritso wants to merge 4 commits intomasterfrom
georgegritsouk/dain-1210-remove-dashboard-templates-feature
Open

ref(dashboards): Remove Dashboard Templates feature#113343
gggritso wants to merge 4 commits intomasterfrom
georgegritsouk/dain-1210-remove-dashboard-templates-feature

Conversation

@gggritso
Copy link
Copy Markdown
Member

@gggritso gggritso commented Apr 17, 2026

Screenshot 2026-04-17 at 2 51 24 PM

Removes the Dashboard Templates feature. With the launch of Prebuilt Dashboards, Dashboard Templates are redundant! They're already hidden for most customers.

Closes DAIN-1210

Dashboard Templates let users browse 4 pre-defined dashboards (General,
Frontend, Backend, Mobile) on the Manage Dashboards page and create new
dashboards from them via a Preview flow. With the launch of Prebuilt
Dashboards, this feature is redundant.

Remove the template selection UI, the template data definitions, the
\`/dashboards/new/:templateId\` route, and the associated analytics
events. \`DashboardState.PREVIEW\` and the \`/dashboards/new/from-seer/\`
flow are kept intact since they are still used by AI-generated
dashboards.

Refs DAIN-1210
Co-Authored-By: Claude <noreply@anthropic.com>
@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 17, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 17, 2026
These tests exercised the Dashboard Templates flow (routes
/dashboards/new/:templateId/... and "save as template from preview")
that was removed in the previous commit. With templates gone, the tests
fail because the template lookup returns undefined.

Remove the three template-specific test cases and the two now-unused
route constants.

Refs DAIN-1210
Co-Authored-By: Claude <noreply@anthropic.com>
Two filter tests waited for '7D' to appear in the page filter header
before interacting with the release dropdown. They passed in CI only
because a prior template test (now removed) clicked through the period
selector and left PageFiltersStore in a matching state. Drop the
precondition check so the tests stand on their own; the subsequent
findBy queries still provide the needed async wait.

Co-Authored-By: Claude <noreply@anthropic.com>
@gggritso gggritso marked this pull request as ready for review April 17, 2026 21:01
@gggritso gggritso requested review from a team as code owners April 17, 2026 21:01
Comment thread static/app/utils/analytics/dashboardsAnalyticsEvents.tsx
The trackAnalytics('dashboards_manage.templates.add') call was still
fired from the PREVIEW/CREATE commit path, but the event key was
removed from the registry in the earlier refactor commit. Drop the
now-dead call. The create flow is still tracked via
'dashboards2.create.complete'.

Co-Authored-By: Claude <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