Skip to content

fix(dashboards): Include hidden dashboards in linked dashboard lookup#110667

Merged
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1339-fix-broken-mobile-vitals-dashboard-detail-links
Mar 13, 2026
Merged

fix(dashboards): Include hidden dashboards in linked dashboard lookup#110667
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1339-fix-broken-mobile-vitals-dashboard-detail-links

Conversation

@DominikB2014
Copy link
Contributor

@DominikB2014 DominikB2014 commented Mar 13, 2026

Add showHidden filter to the prebuilt dashboard ID query in
usePopulateLinkedDashboards so that hidden dashboards (i.e the detail dashboards)
are included when resolving linked dashboard detail links.

Without this filter, the API excludes hidden dashboards from results,
causing detail links for mobile vitals dashboards to break since those
dashboards are marked as hidden.

Fixes DAIN-1339

Add 'showHidden' filter to the prebuilt dashboard ID query so that
hidden mobile vitals dashboards are found when resolving linked
dashboard detail links.

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

linear-code bot commented Mar 13, 2026

@DominikB2014 DominikB2014 marked this pull request as ready for review March 13, 2026 18:55
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 13, 2026
@DominikB2014 DominikB2014 requested a review from a team as a code owner March 13, 2026 18:55
@DominikB2014 DominikB2014 enabled auto-merge (squash) March 13, 2026 18:57
@DominikB2014 DominikB2014 merged commit e39af36 into master Mar 13, 2026
66 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1339-fix-broken-mobile-vitals-dashboard-detail-links branch March 13, 2026 19:02
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