feat(account): BACK-692 show picklist backorder prompts on order details#2683
Open
bc-apostoliuk wants to merge 3 commits into
Open
feat(account): BACK-692 show picklist backorder prompts on order details#2683bc-apostoliuk wants to merge 3 commits into
bc-apostoliuk wants to merge 3 commits into
Conversation
Render per-picklist-option backorder prompts on the account order details page so each backordered bundle option shows its own "N will be backordered" message plus the backorder message, mirroring the PDP bundle backorder display. Item-level backorder prompts are unchanged. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1853cdd. Configure here.
The order details item-level backorder block previously rendered after the options/gift-wrapping/event-date list, so the parent product's backorder prompts (ready to ship / N backordered / message) visually attached to the last bundle option. Move it directly under the product title so it is clearly attributed to the parent product, with per-picklist-option backorder prompts still rendered under each option — matching the PDP layout where the main product and each bundle show their backorder status separately. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The storefront order object exposes only name and value per picklist option (verified against a live bundle order) — no per-option quantity_backordered or backorder_message. The per-option backorder block could therefore never render, so remove it. Parent-product backorder prompts under the product title remain the only backorder data the order payload supports. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

What?
Renders backorder prompts per picklist (bundle) option on the account order details page (
templates/components/account/order-contents.html), so each backordered bundle option displays its own backorder status — mirroring how the PDP renders per-bundle backorder.Previously each option only rendered its name/value via the
product-optionspartial. Now, inside the{{#each options}}loop, when an option hasquantity_backordered > 0it additionally renders:{quantity} will be backordered(via theproducts.quantity_backorderedlang string)backorder_message, when presentThis matches the PDP bundle backorder display (
assets/js/theme/common/picklist-backorder.js), which shows only the backordered quantity and message per option — there is intentionally no per-option "ready to ship" line. The existing item-level backorder block (ready to ship + backordered + message for the whole line item) is unchanged.Technical notes for reviewers:
<dt>key, so the combined output reads like the PDP's"<Option name>: N will be backordered"+ message.quantity_on_hand("ready to ship") line using(subtract quantity quantity_backordered). That was removed: the PDP doesn't show it per option, and inside{{#each options}}thequantitytoken resolves against the option context (not the line item), so the math was unreliable.quantity_backordered/backorder_messageper option in the order object. The item-level fields are confirmed present; the per-option fields could not be verified locally because the only available test order has no bundle options. Reviewers/QA should confirm against an order containing a backordered picklist option.Requirements
Tickets / Documentation
Screenshots (if appropriate)
To be added — pending an order containing a backordered picklist (bundle) option.
🤖 Generated with Claude Code
Note
Low Risk
Presentation-only Handlebars reordering using existing order line fields; no auth, checkout, or cart logic changes.
Overview
Account order details now shows line-item backorder messaging (ready-to-ship quantity when applicable, backordered count, and optional
backorder_message) directly under the product title/brand, instead of below gift-wrapping and event-date fields.The Draft CHANGELOG entry documents showing backorder prompts on this page for the parent line and picklist options, aligned with PDP backorder display.
Reviewed by Cursor Bugbot for commit 921b456. Bugbot is set up for automated code reviews on this repo. Configure here.