Skip to content

fix(emails): Subscription invoice emails have outdate email addresses#19498

Merged
david1alvarez merged 1 commit intomainfrom
PAY-3215
Sep 23, 2025
Merged

fix(emails): Subscription invoice emails have outdate email addresses#19498
david1alvarez merged 1 commit intomainfrom
PAY-3215

Conversation

@david1alvarez
Copy link
Copy Markdown
Contributor

Because:

  • When a customer changes their FxA account email, this change is not propagated through to Stripe
  • Emails relating to Stripe's invoices pull the customer's email field from their invoice
  • This causes these emails to contain outdated email addresses

This commit:

  • Because the text of the invoice references the customer receiving the email because there is a Mozilla account with the listed email address, this commit updates the emails to use the account's primary email for these fields.

Closes #PAY-3215

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).

Screenshots (Optional)

Prior to updating my email address via FxA Account Settings:
Screenshot 2025-09-23 at 12 46 32 PM

After updating my email address
Screenshot 2025-09-23 at 12 47 10 PM

Other information (Optional)

Any other information that is important to this pull request.

@david1alvarez david1alvarez requested a review from a team as a code owner September 23, 2025 19:47
Because:

* When a customer changes their FxA account email, this change is not propagated through to Stripe
* Emails relating to Stripe's invoices pull the customer's email field from their invoice
* This causes these emails to contain outdated email addresses

This commit:

* Because the text of the invoice references the customer receiving the email because there is a Mozilla account with the listed email address, this commit updates the emails to use the account's primary email for these fields.

Closes #PAY-3215
Copy link
Copy Markdown
Contributor

@xlisachan xlisachan left a comment

Choose a reason for hiding this comment

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

Confirming that I was able to see the new primary email address in the downloaded invoice when I upgraded a subscription and created a new subscription.

r+, thanks Davey!

@david1alvarez david1alvarez merged commit ab9011f into main Sep 23, 2025
19 checks passed
@david1alvarez david1alvarez deleted the PAY-3215 branch September 23, 2025 22:31
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