Skip to content
This repository has been archived by the owner on May 5, 2022. It is now read-only.

Remove duplicated breadcrumb items #117

Merged
merged 1 commit into from
Dec 5, 2019
Merged

Remove duplicated breadcrumb items #117

merged 1 commit into from
Dec 5, 2019

Conversation

ilyawzrd
Copy link
Contributor

@ilyawzrd ilyawzrd commented Dec 4, 2019 •
edited
Loading

for #115
fix1
fix2

@ilyawzrd ilyawzrd requested a review from yecli December 4, 2019 19:29
@ilyawzrd
Copy link
Contributor Author

ilyawzrd commented Dec 5, 2019

Part of changes from this commit 6f6947e to breadcrumb.liquid was unnecessary. Elsif syntax is working in scriban and we have the same implementation in b2b-theme right now with correct behavior

@yecli yecli requested a review from tatarincev December 5, 2019 09:26
@yecli yecli merged commit c4ff078 into dev Dec 5, 2019
@yecli yecli deleted the 1072-remove-duplicated-breadcrumbs branch December 5, 2019 14:17
@akak1977
Copy link
Contributor

Released https://github.com/VirtoCommerce/vc-theme-default/releases/tag/v2.0.4

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@yecli yecli yecli approved these changes

@tatarincev tatarincev Awaiting requested review from tatarincev

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ilyawzrd @akak1977 @yecli