Skip to content

feat(orders): ORDERS-7734 add template for guest return portal#2645

Open
BC-SEven wants to merge 1 commit into
feature/PROJECT-7525-new-returns-flowfrom
ORDERS-7734
Open

feat(orders): ORDERS-7734 add template for guest return portal#2645
BC-SEven wants to merge 1 commit into
feature/PROJECT-7525-new-returns-flowfrom
ORDERS-7734

Conversation

@BC-SEven
Copy link
Copy Markdown

@BC-SEven BC-SEven commented Apr 28, 2026

What?

Add new guest return portal template.

See companion PRs:

Requirements

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

Tickets / Documentation

  • ORDERS-7734

Screenshots (if appropriate)

See https://github.com/bigcommerce/bigcommerce/pull/67869
Example Image

@BC-SEven BC-SEven changed the base branch from master to feature/PROJECT-7525-new-returns-flow May 4, 2026 01:12
@BC-SEven BC-SEven changed the title [Spike] [DO NOT MERGE] feat(orders): ORDERS-7734 add template for guest return portal feat(orders): ORDERS-7734 add template for guest return portal May 4, 2026
@BC-SEven BC-SEven marked this pull request as ready for review May 11, 2026 00:53
@BC-SEven BC-SEven requested a review from a team as a code owner May 11, 2026 00:53
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

For my understanding - html page name must always match router path to work correctly?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Yep the file name and path here needs to match whats in the storefront routing

case object GuestReturnPortalRoute extends StorefrontTemplateRoute("guest_return_portal", "pages/guest-return-portal")

{{#partial "page"}}

{{> components/common/breadcrumbs breadcrumbs=breadcrumbs}}

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.

I think we could remove the extra lines (4 and 7) and add a new one empty at 12
Also update the CHANGELOG.md please.

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.

4 participants