Skip to content

fix(payments-next): Subscription reminder emails showing exclusive tax for non-US customers#20298

Open
elizabeth-ilina wants to merge 1 commit intomainfrom
PAY-3601-subscription-reminder-emails-showing-exlusive-tax-for-non-us-customers
Open

fix(payments-next): Subscription reminder emails showing exclusive tax for non-US customers#20298
elizabeth-ilina wants to merge 1 commit intomainfrom
PAY-3601-subscription-reminder-emails-showing-exlusive-tax-for-non-us-customers

Conversation

@elizabeth-ilina
Copy link
Copy Markdown
Contributor

@elizabeth-ilina elizabeth-ilina commented Mar 31, 2026

Because

  • In the subscription renewal reminder emails, European customers are being shown the prices as “€(base amount) + €(tax amount) tax”. This is a pattern that should only be shown to US customers.

This pull request

  • Uses Stripe's total_tax_amounts[].inclusive to determine whether to show the customer a single total or a breakdown of base amount + tax.

Issue that this pull request solves

Closes #PAY-3601

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).
  • I have manually reviewed all AI generated code.

How to review (Optional)

  • Key files/areas to focus on: packages/fxa-auth-server/lib/payments/subscription-reminders.ts
  • Suggested review order:
  • Risky or complex parts:

Screenshots (Optional)

  • Usd email:
image
  • Eur:
image

Other information (Optional)

Any other information that is important to this pull request.

@elizabeth-ilina elizabeth-ilina force-pushed the PAY-3601-subscription-reminder-emails-showing-exlusive-tax-for-non-us-customers branch 4 times, most recently from d5dc27d to 3453c9e Compare March 31, 2026 21:28
…x for non-US customers

Because:

* In the subscription renewal reminder emails, European customers are being shown the prices as “€(base amount) + €(tax amount) tax”. This is a pattern that should only be shown to US customers.

This commit:

* Uses Stripe's total_tax_amounts[].inclusive to determine whether to show the customer a single total or a breakdown of base amount + tax.

Closes #[PAY-3601](https://mozilla-hub.atlassian.net/browse/PAY-3601)
@elizabeth-ilina elizabeth-ilina force-pushed the PAY-3601-subscription-reminder-emails-showing-exlusive-tax-for-non-us-customers branch from 3453c9e to e461934 Compare March 31, 2026 21:29
@elizabeth-ilina elizabeth-ilina marked this pull request as ready for review April 1, 2026 11:33
@elizabeth-ilina elizabeth-ilina requested a review from a team as a code owner April 1, 2026 11:33
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.

1 participant