Skip to content

[_]: fix/checkout-tax-flash-flash-on-billing-country-refetch - #2103

Merged
jaaaaavier merged 1 commit into
masterfrom
fix/checkout-tax-flash
Aug 6, 2026
Merged

[_]: fix/checkout-tax-flash-flash-on-billing-country-refetch#2103
jaaaaavier merged 1 commit into
masterfrom
fix/checkout-tax-flash

Conversation

@sg-gs

@sg-gs sg-gs commented Aug 6, 2026

Copy link
Copy Markdown
Member

What

Restores postalCode in the debounced price refetch triggered by billing address changes in CheckoutViewWrapper, and guards the effect so it only fires once both country and postal code are known.

Why

Since 9cc570d47 ("checkout fixes"), billingCountry is populated immediately from IP geolocation on page load (before the user enters anything), which fires a refetch of the price ~500ms later using only country (no postalCode, no userAddress). That refetch returns a different tax amount than the initial fetch, so the tax line item flashes on screen and then disappears. The actual charge is unaffected (postal code is read from the Stripe confirmation token at payment time), so this was a display-only regression.

@sg-gs
sg-gs requested a review from xabg2 as a code owner August 6, 2026 11:36
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: a773b23
Status: ✅  Deploy successful!
Preview URL: https://4e4e97fb.drive-web.pages.dev
Branch Preview URL: https://fix-checkout-tax-flash.drive-web.pages.dev

View logs

@sonarqubecloud

sonarqubecloud Bot commented Aug 6, 2026

Copy link
Copy Markdown

@sg-gs sg-gs self-assigned this Aug 6, 2026
@jaaaaavier
jaaaaavier merged commit 69d35e8 into master Aug 6, 2026
11 checks passed
@jaaaaavier
jaaaaavier deleted the fix/checkout-tax-flash branch August 6, 2026 11:43
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.

2 participants