Added fees and duties to cart, checkout and order totals - #217
damianlegawiec wants to merge 5 commits into
Conversation
Adds an opt-in link to a local @spree/sdk build so the storefront can be developed against unreleased Store API features. Without SPREE_SDK_PATH the install is unchanged, keeping standalone clones and CI on the published SDK.
Without it pnpm walks up the filesystem and adopts an enclosing workspace, installing that project's dependencies and overrides here instead of these.
Module resolution is pinned to this project's own directory, so a package symlinked outside it fails to resolve and every import of @spree/sdk breaks.
The dependency is copied rather than symlinked, so a rebuild needs a reinstall to take effect.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Strix Security ReviewNo security issues found. Updated for Reviewed by Strix |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Spree 6.0 carts and orders carry a fees array (gift wrapping, handling, cash on delivery, payment surcharges, customs duties). The storefront never rendered them, so a fee changed the total with no line explaining why. Each fee now shows as its own line, under its own label, on the cart page, cart drawer, checkout summary, order totals and the order confirmation email. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
1b3161a to
16ef075
Compare
1 participant
Spree 6.0 carts and orders carry a
feesarray (gift wrapping, handling, cash on delivery, payment surcharges, customs duties). The storefront never rendered them, so a fee changed the total with no line explaining why. Cross-border shoppers in particular expect to see an import duty called out before they pay.total, so they are listed between the subtotal and the total and nothing is added on top🤖 Generated with Claude Code