Skip to content

fix(AppShell): update breadcrumbs on well pdf preview page#307

Merged
TylerAdamMartinez merged 1 commit into
stagingfrom
fix/tam-BDMS-938
Jun 26, 2026
Merged

fix(AppShell): update breadcrumbs on well pdf preview page#307
TylerAdamMartinez merged 1 commit into
stagingfrom
fix/tam-BDMS-938

Conversation

@TylerAdamMartinez

@TylerAdamMartinez TylerAdamMartinez commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Why

This PR addresses the following problem/context:

  • Improve breadcrumb navigation for PDF preview pages.
  • Eliminate duplicate breadcrumb rendering by using the shared header breadcrumb component.

How

Implementation summary - the following was changed/added/removed:

  • Extended the breadcrumb route pattern to recognize pdf-preview routes in addition to show and edit.
  • Removed the page-specific AppBreadcrumb from the Well PDF Preview page

Notes

Any special considerations, workarounds, or follow-up work to note?

  • Breadcrumb behavior for existing Show and Edit pages remains unchanged.

Before

Captura de pantalla 2026-06-26 a la(s) 13 16 13

After

Captura de pantalla 2026-06-26 a la(s) 13 17 07

@TylerAdamMartinez TylerAdamMartinez self-assigned this Jun 26, 2026
@TylerAdamMartinez TylerAdamMartinez added the bug Something isn't working label Jun 26, 2026
@TylerAdamMartinez TylerAdamMartinez marked this pull request as ready for review June 26, 2026 18:21
@github-actions

Copy link
Copy Markdown

Preview Deployment

Preview URL: https://preview-fix-tam-bdms-938-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@jeremyzilar jeremyzilar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice work!

@TylerAdamMartinez TylerAdamMartinez merged commit 6920d4a into staging Jun 26, 2026
4 of 5 checks passed
@TylerAdamMartinez TylerAdamMartinez deleted the fix/tam-BDMS-938 branch June 26, 2026 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants