Skip to content

fix(sidebar): highlight conversations tab on nested conversation/transcript pages#708

Open
dembrane-sam-bot wants to merge 2 commits into
mainfrom
sam/highlight-sidebar-conversations-tab
Open

fix(sidebar): highlight conversations tab on nested conversation/transcript pages#708
dembrane-sam-bot wants to merge 2 commits into
mainfrom
sam/highlight-sidebar-conversations-tab

Conversation

@dembrane-sam-bot

@dembrane-sam-bot dembrane-sam-bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

What this changes

  • Aligned the nested conversation detail route path prefix (/conversations/:conversationId) with the list route path (/conversations), ensuring standard matching rules organically highlight the "Conversations" sidebar tab on all detail, transcript, and sub-pages without requiring custom force-active logic — Router.tsx, ProjectHomeView.tsx
  • Refactored all internal links, redirection paths, breadcrumbs, search result navigations, and example webhook payloads across the frontend to use the updated /conversations/:conversationId path structure — 17 files modified (full list in diff)

Confidence

High. This is a very clean architecture-level alignment of the path prefix hierarchy that resolves the highlighting issue automatically using standard React Router configuration. Tested matching semantics and reference updates.

@github-actions

Copy link
Copy Markdown

Hi @dembrane-sam-bot!

Thank you for contributing to Dembrane ECHO! Before we consider your Pull Request, we ask that you sign our Contributor License Agreement (CLA). This is only required for your first Pull Request.

Please review the CLA, and sign it by adding your GitHub username to the contributors.yml file. Thanks!

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