Problem/Motivation

Incorrect breadcrumb generation on excluding middle level of the breadcrumb segment via "Paths to be excluded while generating segments" field

Steps to reproduce

1. Create multi-level breadcrumb (e.g /level-1/level-2/level-3/level-4)
2. In easy breadcrumb configuration, exclude middle level segment (e.g level-3) via "Paths to be excluded while generating segments" field
3. Notice level-1 and level-2 also getting excluded from breadcrumb generation along with level-3

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 incorrect_breadcrumb_generation-3169128-3.patch599 bytesankit agrawal

Comments

Ankit Agrawal created an issue. See original summary.

ankit agrawal’s picture

ankit agrawal commented
Assigned: ankit agrawal » Unassigned
ankit agrawal’s picture

ankit agrawal commented
Status: Active » Needs review
StatusFileSize
new incorrect_breadcrumb_generation-3169128-3.patch599 bytes

Created a patch to fix this issue, please review.

Thanks!

greg boggs’s picture

greg boggs
he/him
Primary language English
Location Portland Oregon
commented

I think there are two different understandings of the same feature! So, let's do both, and clarify the admin UI.

1. Exclude a path from breadcrumbs so that path doesn't get crumbs.

2. Exclude a path from breadcrumbs so that path doesn't appear on crumbs displayed on other paths.

4everzip’s picture

4everzip commented

#3 worked for me
Thanks

neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented
Version: 8.x-1.13 » 8.x-1.x-dev
greg boggs’s picture

greg boggs
he/him
Primary language English
Location Portland Oregon
commented
Status: Needs review » Reviewed & tested by the community

neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.