Skip to content

Conversation

@narsaynorath
Copy link
Member

@narsaynorath narsaynorath commented Jan 15, 2026

The navigate call here was triggering a failed attempt to Add to Dashboard for adding All Metrics. The reason why it does this is because the route components are lazily loaded. When the first view occurs, it lazily loads the component. When the navigate is called, it goes down a different code path which seems to remount the component.

Getting rid of the navigate call fixes this issue and location.state isn't preserved through navigations so there isn't a need to reset the state.

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 15, 2026
@narsaynorath narsaynorath changed the title WIP: fix(dashboards): Add to dashboard fails to add on first attempt WIP: fix(dashboards): Add All Metrics fails to add on first attempt Jan 15, 2026
@narsaynorath narsaynorath changed the title WIP: fix(dashboards): Add All Metrics fails to add on first attempt fix(dashboards): Add All Metrics fails to add on first attempt Jan 15, 2026
@narsaynorath narsaynorath marked this pull request as ready for review January 15, 2026 16:57
@narsaynorath narsaynorath requested a review from a team as a code owner January 15, 2026 16:57
@narsaynorath narsaynorath merged commit 50d2e99 into master Jan 15, 2026
56 checks passed
@narsaynorath narsaynorath deleted the nar/fix/dashboards-add-to-dashboard-fails-to-add-on-first-attempt branch January 15, 2026 17:06
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.

3 participants