Skip to content

use sku currency codes in storefront components#1386

Open
pbennett1-godaddy wants to merge 1 commit into
mainfrom
storefont-currency-fixes
Open

use sku currency codes in storefront components#1386
pbennett1-godaddy wants to merge 1 commit into
mainfrom
storefont-currency-fixes

Conversation

@pbennett1-godaddy

Copy link
Copy Markdown
Collaborator

Summary

Fix storefront currency formatting so product prices use the currency returned by catalog SKU pricing instead of hardcoded USD.

SKUGroup.priceRange only provides min / max, but SKU prices include value.currencyCode. Product card and product details now read currency from SKU price data and only fall back to USD when
no currency is available. Cart creation also no longer sends hardcoded zero-value USD totals to addDraftOrder, since totals are now calculated by the order API from line items.

Changeset

  • Changeset added (docs)

Test Plan

  • Ran Biome checks on changed storefront files:

    • pnpm --filter @godaddy/react exec biome check src/components/storefront/product-card.tsx src/lib/godaddy/catalog-storefront-queries.ts
    • pnpm --filter @godaddy/react exec biome check src/components/storefront/hooks/use-add-to-cart.ts
    • pnpm --filter @godaddy/react exec biome check src/components/storefront/product-details.tsx
  • Ran typecheck:

    • pnpm --filter @godaddy/react typecheck

@pbennett1-godaddy pbennett1-godaddy requested a review from a team as a code owner June 15, 2026 14:01
@changeset-bot

changeset-bot Bot commented Jun 15, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 07574f2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@godaddy/react Patch
nextjs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pagrawal1-godaddy pagrawal1-godaddy left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

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.

3 participants