Skip to content

feat(payments-next): subscriptionRenewalReminder-content-charge uses hard-coded English values for subscription interval#19989

Open
elizabeth-ilina wants to merge 1 commit intomainfrom
PAY-3497-subscription-renewal-reminder-content-charge-uses-hard-coded-english-values-for-subscription-interval
Open

feat(payments-next): subscriptionRenewalReminder-content-charge uses hard-coded English values for subscription interval#19989
elizabeth-ilina wants to merge 1 commit intomainfrom
PAY-3497-subscription-renewal-reminder-content-charge-uses-hard-coded-english-values-for-subscription-interval

Conversation

@elizabeth-ilina
Copy link
Contributor

@elizabeth-ilina elizabeth-ilina commented Feb 3, 2026

Because

  • The subscriptionRenewalReminder-content-charge fluent string has a variable hardcoded in English, so a localized string will display with a mix of English and translated text.

This pull request

  • Updates the string for proper localization and to match existing subscription plan lengths (daily, weekly, monthly, halfyearly (6-monthly), yearly.
  • Updates the string to break out the charge to show the tax amount (where applicable).

Issue that this pull request solves

Closes: #PAY-3497

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)

Please attach the screenshots of the changes made in case of change in user interface.
With tax:
image

No Tax:
image

Other information (Optional)

Any other information that is important to this pull request.

@elizabeth-ilina elizabeth-ilina force-pushed the PAY-3497-subscription-renewal-reminder-content-charge-uses-hard-coded-english-values-for-subscription-interval branch 5 times, most recently from b5a9273 to c2c205c Compare February 5, 2026 16:35
…hard-coded English values for subscription interval

Because:

* The subscriptionRenewalReminder-content-charge fluent string has a variable hardcoded in English, so a localized string will display with a mix of English and translated text.

This commit:

* Updates the string for proper localization and to match existing subscription plan lengths (daily, weekly, monthly, halfyearly (6-monthly), yearly.
* Updates the string to break out the charge to show the tax amount (where applicable). - TODO

Closes #[PAY-3497](https://mozilla-hub.atlassian.net/browse/PAY-3497)
@elizabeth-ilina elizabeth-ilina force-pushed the PAY-3497-subscription-renewal-reminder-content-charge-uses-hard-coded-english-values-for-subscription-interval branch from c2c205c to dd66115 Compare February 5, 2026 17:41
@elizabeth-ilina elizabeth-ilina marked this pull request as ready for review February 5, 2026 17:42
@elizabeth-ilina elizabeth-ilina requested review from a team as code owners February 5, 2026 17:42
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