Skip to content

feat(orders): ORDERS-7647 update return list navigation to work with …#2714

Merged
BC-SEven merged 1 commit into
feature/PROJECT-7525-new-returns-flowfrom
ORDERS-7647-return-list-navigation
Jul 23, 2026
Merged

feat(orders): ORDERS-7647 update return list navigation to work with …#2714
BC-SEven merged 1 commit into
feature/PROJECT-7525-new-returns-flowfrom
ORDERS-7647-return-list-navigation

Conversation

@BC-SEven

@BC-SEven BC-SEven commented Jul 23, 2026

Copy link
Copy Markdown

…v2 returns

What?

Update logic for displaying return navigation to also display for v2 returns setting.

Requirements

  • CHANGELOG.md entry added (required for code changes only)

Tickets / Documentation

NA

Screenshots (if appropriate)

When settings enabled
image

Returns listed in accounts nav

image

Note

Low Risk
Template-only visibility change for existing nav links; no new URLs or auth logic.

Overview
Returns navigation now appears when either legacy settings.returns_enabled or settings.returns_v2_enabled is on, instead of only the legacy flag.

The same {{#or ...}} guard is applied in the account sub-nav (navigation.html) and the mobile/storefront account submenu (navigation-menu.html), so shoppers with v2 returns enabled can reach the returns list without turning on the old setting. CHANGELOG documents the change.

Reviewed by Cursor Bugbot for commit 2bd0137. Bugbot is set up for automated code reviews on this repo. Configure here.

@BC-SEven
BC-SEven marked this pull request as ready for review July 23, 2026 07:05
@BC-SEven
BC-SEven requested a review from a team as a code owner July 23, 2026 07:05
@BC-SEven
BC-SEven force-pushed the ORDERS-7647-return-list-navigation branch from 8b9f638 to 2bd0137 Compare July 23, 2026 07:07
@BC-SEven
BC-SEven merged commit 8447deb into feature/PROJECT-7525-new-returns-flow Jul 23, 2026
8 checks passed
@BC-SEven
BC-SEven deleted the ORDERS-7647-return-list-navigation branch July 23, 2026 23:39
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.

2 participants