Skip to content

[Issue #8861] Improve /saved-opportunities breadcrumbs#9018

Open
andycochran wants to merge 1 commit intomainfrom
andycochran/8861--saved-op-breadcrumbs
Open

[Issue #8861] Improve /saved-opportunities breadcrumbs#9018
andycochran wants to merge 1 commit intomainfrom
andycochran/8861--saved-op-breadcrumbs

Conversation

@andycochran
Copy link
Collaborator

@andycochran andycochran commented Mar 12, 2026

Summary

Fixes #8861

Changes proposed

  • Delete SAVED_OPPORTUNITIES_CRUMBS constant
  • Define breadcrumbs inline in the page where they're used
image

Context for reviewers

Per new guidance, all breadcrumbs are to be managed consistently, i.e. on their page (not in the constant file). This is part of a larger effort to refactor breadcrumbs. In a subsequent ticket, /saved-opportunities will be moved to /workspace/saved-opportunities. But that'll happen along with other route changes, when /dashboard will also change to /workspace.

Validation steps

  • Sign in
  • View the /saved-opportunities page
  • The breadcrumbs should read "Workspace > Saved opportunities"
  • The "Workspace" breadcrumb should take you to /dashboard

@andycochran andycochran marked this pull request as ready for review March 12, 2026 19:31
@andycochran andycochran requested review from btabaska, doug-s-nava, mdragon and myduong-navapbc and removed request for doug-s-nava March 12, 2026 19:31
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.

Improve "Saved opportunities" breadcrumbs

1 participant