Skip to content

[Improve] Hide PR analytics from the sidebar - #891

Merged
mrubens merged 1 commit into
developfrom
fix/hide-pr-analytics-link-06rkii44bwidd
Jul 29, 2026
Merged

[Improve] Hide PR analytics from the sidebar#891
mrubens merged 1 commit into
developfrom
fix/hide-pr-analytics-link-06rkii44bwidd

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

Opened on behalf of Matt Rubens. Follow up by mentioning @roomote-roomote, in the web UI, or in Slack.

What changed

Removed the PR analytics entry from the Analytics sidebar while retaining the existing PR analytics route for direct access.

Updated the focused navigation test to cover the hidden entry and preserved route behavior.

Why this change was made

The PR analytics surface should remain out of primary navigation until it is ready to be promoted again.

Impact

Users now see only Tasks and Costs in the Analytics sidebar. Existing direct links to PR analytics continue to work.

Screenshots

Analytics navigation showing Tasks and Costs with no PRs entry

@roomote-roomote

roomote-roomote Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor Author

1 issue outstanding. See task

  • apps/web/src/app/(authenticated)/analytics/AnalyticsShell.tsx:60-63: The direct ?object=pullRequests view now falls back to tasks as its active item. The desktop sidebar highlights Tasks and the mobile selector displays Tasks while the page title and content are PRs, which misrepresents the current page. Keep the hidden PR view unselected instead and cover that state in the navigation test.

Reviewed db8d4f2

@mrubens
mrubens marked this pull request as ready for review July 29, 2026 22:20
@mrubens
mrubens merged commit e961a2e into develop Jul 29, 2026
17 checks passed
@mrubens
mrubens deleted the fix/hide-pr-analytics-link-06rkii44bwidd branch July 29, 2026 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant