Skip to content

fix: Fix PathwayBrowser navigation and event hierarchy tree#28

Merged
el-rabies merged 1 commit intomainfrom
fix-pathway-tree-navigation
Mar 16, 2026
Merged

fix: Fix PathwayBrowser navigation and event hierarchy tree#28
el-rabies merged 1 commit intomainfrom
fix-pathway-tree-navigation

Conversation

@adamjohnwright
Copy link
Contributor

Use route-relative navigation (navigateTo) instead of router.navigate across all PathwayBrowser components to ensure URLs stay under the /PathwayBrowser/ prefix in the umbrella app. Add missing MatTreeNodeToggle and MatTreeNodeOutlet imports to event-hierarchy component so tree expand/collapse and child rendering work correctly. Fix _ignore flag getting stuck and preserve expansion state across data source resets.

Use route-relative navigation (navigateTo) instead of router.navigate
across all PathwayBrowser components to ensure URLs stay under the
/PathwayBrowser/ prefix in the umbrella app. Add missing MatTreeNodeToggle
and MatTreeNodeOutlet imports to event-hierarchy component so tree
expand/collapse and child rendering work correctly. Fix _ignore flag
getting stuck and preserve expansion state across data source resets.

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

Hi! I'm the autofix logoautofix.ci troubleshooter bot.

It looks like you correctly set up a CI job that uses the autofix.ci GitHub Action, but the autofix.ci GitHub App has not been installed for this repository. This means that autofix.ci unfortunately does not have the permissions to fix this pull request. If you are the repository owner, please install the app and then restart the CI workflow! 😃

Copy link
Collaborator

@el-rabies el-rabies left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@el-rabies el-rabies merged commit 8297673 into main Mar 16, 2026
1 of 2 checks passed
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.

3 participants