Skip to content

Bugfix: Repeated breadcrumb issue#60

Merged
Garneauma merged 5 commits intoServiceCanada:mainfrom
cfoss-coveo:bugfix/427242-sr-604-canadaca-breadcrumb-issue
Apr 16, 2026
Merged

Bugfix: Repeated breadcrumb issue#60
Garneauma merged 5 commits intoServiceCanada:mainfrom
cfoss-coveo:bugfix/427242-sr-604-canadaca-breadcrumb-issue

Conversation

@cfoss-coveo
Copy link
Copy Markdown
Contributor

Tickets/Tasks:

Description:

  • Addressed an issue where the hostname is duplicated for certain items in the result breadcrumbs

Cause:

  • On items with a single breadcrumb pointed to the home page, we replace the scraped breadcrumb with a link to the domain root. .i.e., <a href="https://www.canada.ca">www.canada.ca</a>.
  • In the front end search result template, when we build the breadcrumbs, we use the hostname + last breadcrumb.
  • This resulted in the breadcrumbs being repeated on these items. i.e., canada.ca > www.canada.ca

Testing:

  • You can find affected items by searching for @wordcount==1 in the Content Browser. Identify an item, then try to bring it up on the search results. Before, you'd see the address repeated: bst-tsb.gc.ca > www.bst-tsb.gc.ca. Now, you'll just see the hostname: bst-tsb.gc.ca
  • Sample searches:
    • "Change your address government/change-address.html"
    • "Other building finishing contractors - 23839 - Financial performance"
    • "Mycobacterium tuberculosis complex and non-tuberculous mycobacteria"
    • "Webinar transcript: Due diligence in the garment industry"
image image

@cfoss-coveo cfoss-coveo requested a review from a team March 5, 2026 18:47
Yves-Denis
Yves-Denis previously approved these changes Mar 5, 2026
Copy link
Copy Markdown

@Yves-Denis Yves-Denis left a comment

Choose a reason for hiding this comment

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

Security Assessment Completed - Passed, no issues identified

Comment thread src/connector.js Outdated
Comment thread src/connector.js Outdated
Co-authored-by: Francis Gorman <francis.gorman25@gmail.com>
Co-authored-by: Francis Gorman <francis.gorman25@gmail.com>
@cfoss-coveo cfoss-coveo requested a review from GormFrank March 19, 2026 22:26
@GormFrank GormFrank requested a review from ipaksc April 9, 2026 19:21
@Garneauma Garneauma added this to the v1.8.1 milestone Apr 9, 2026
Copy link
Copy Markdown
Contributor

@Garneauma Garneauma left a comment

Choose a reason for hiding this comment

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

@cfoss-coveo Tested locally with a few sample searches. It does remove the duplication of breadcrumb items compared to the live search on Canada.ca.

Only one small technical change. Thanks!

Comment thread src/connector.js Outdated
Moved the `ol` tag outside the breadcrumb conditional

Co-authored-by: Garneauma <garneauma@gmail.com>
@Garneauma Garneauma requested a review from Yves-Denis April 16, 2026 14:36
Copy link
Copy Markdown

@Yves-Denis Yves-Denis left a comment

Choose a reason for hiding this comment

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

Further Security review completed - no issues identified.

Copy link
Copy Markdown
Contributor

@ipaksc ipaksc left a comment

Choose a reason for hiding this comment

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

Tested in production using a test page. Everything works as expected.

@Garneauma Garneauma merged commit cc71d47 into ServiceCanada:main Apr 16, 2026
1 check 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.

5 participants