-
Notifications
You must be signed in to change notification settings - Fork 110
Fix link for configurable products and update account version #994
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
📝 Block Changes DetectedThis PR contains changes to the following blocks:
📋 README RequirementsPlease ensure the README.md for each changed block includes:
✅ All blocks have README filesPlease review and update the README files to ensure they accurately reflect any changes made to the blocks. This comment was automatically generated by the Block README Check workflow. |
…commerce into USF-3576
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates the storefront-account package from beta4 to beta5 to fix product linking for configurable and bundle products in order history. The change adds a new topLevelSku field to order items and updates the license agreement with new terms for Adobe Commerce as a Cloud Service and Adobe Commerce Optimizer.
Key Changes:
- Added
topLevelSkufield to order history items for proper product linking - Updated license agreement to clarify platform restrictions and fix typos
- Modified product link generation to use
topLevelSkuinstead of variantsku
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| scripts/dropins/storefront-account/chunks/getOrderHistoryList.js | Added topLevelSku field extraction from product data in order history query |
| scripts/dropins/storefront-account/LICENSE.md | Updated license terms, corrected typos, and added platform restrictions |
| package.json | Bumped @dropins/storefront-account version from beta4 to beta5 |
| blocks/commerce-orders-list/commerce-orders-list.js | Updated to use topLevelSku instead of sku for product link generation |
| blocks/commerce-order-product-list/commerce-order-product-list.js | Added validation and changed to use product.sku for link generation |
blocks/commerce-order-product-list/commerce-order-product-list.js
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
bl4de
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is to fix the configurable and bundle product links and the account package version updated with the code needed for the fix and the new license
Fix USF-3576
Test URLs: