Skip to content

feat(payments): Switch all Mozilla accounts legal links over to Mozilla.org version - #20911

Open
elizabeth-ilina wants to merge 1 commit into
mainfrom
PAY-3829-switch-all-mozilla-accounts-legal-links
Open

feat(payments): Switch all Mozilla accounts legal links over to Mozilla.org version#20911
elizabeth-ilina wants to merge 1 commit into
mainfrom
PAY-3829-switch-all-mozilla-accounts-legal-links

Conversation

@elizabeth-ilina

@elizabeth-ilina elizabeth-ilina commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Because

This pull request

Issue that this pull request solves

Closes #PAY-3829

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:
  • Suggested review order:
  • Risky or complex parts:

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

Other information (Optional)

Any other information that is important to this pull request.

@elizabeth-ilina
elizabeth-ilina force-pushed the PAY-3829-switch-all-mozilla-accounts-legal-links branch 4 times, most recently from 2ee6138 to d4d7181 Compare July 23, 2026 14:10
Comment thread apps/payments/next/next-env.d.ts Outdated
@elizabeth-ilina
elizabeth-ilina force-pushed the PAY-3829-switch-all-mozilla-accounts-legal-links branch from d4d7181 to f8ad3b6 Compare July 23, 2026 17:41
@elizabeth-ilina
elizabeth-ilina requested a review from a team July 23, 2026 17:45
@elizabeth-ilina
elizabeth-ilina force-pushed the PAY-3829-switch-all-mozilla-accounts-legal-links branch 2 times, most recently from 835f21b to d85ead3 Compare July 27, 2026 18:36
@elizabeth-ilina
elizabeth-ilina marked this pull request as ready for review July 28, 2026 15:03
@elizabeth-ilina
elizabeth-ilina requested a review from a team as a code owner July 28, 2026 15:03
@elizabeth-ilina
elizabeth-ilina force-pushed the PAY-3829-switch-all-mozilla-accounts-legal-links branch 3 times, most recently from 925db37 to 10ff5f5 Compare July 31, 2026 16:30
…la.org version

Because:

* We use Mozilla account links (e.g. https://accounts.firefox.com/legal/terms) throughout the repo.

This commit:

* Updates links for all accounts & subplat surfaces to point to current versions on http://mozilla.org.
* Instead of downloadable PDFs for legal docs in SubPlat emails, replaces the links to download legal PDFs to the equivalent legal docs on http://mozilla.org (e.g., …/subscription_services_privacy_notice.pdf is replaced by https://www.mozilla.org/privacy/subscription-services/)
* Retires https://github.com/mozilla/fxa/actions/workflows/pull-legal-docs.yml github action
* Does NOT remove current Legal docs from the accounts CDN, since these need to stay in place for emails sent prior to this change to avoid broken URLs.

Closes #[PAY-3829](https://mozilla-hub.atlassian.net/browse/PAY-3829)
@elizabeth-ilina
elizabeth-ilina force-pushed the PAY-3829-switch-all-mozilla-accounts-legal-links branch from 10ff5f5 to e860e37 Compare July 31, 2026 17:05

@StaberindeZA StaberindeZA left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I noticed in the FxA sign up email that the one Privacy link had query params, but the terms one didn't. Is this related to any of these changes?

Image

  •  

<a href="<%- subscriptionTermsUrl %>" class="footer-link" data-l10n-id="subplat-moz-terms">Mozilla Accounts Terms of Service</a>
<a href="https://www.mozilla.org/about/legal/terms/services/" class="footer-link" data-l10n-id="subplat-moz-terms">Mozilla Accounts Terms of Service</a>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

question: why hardcode the value instead of using the variable?

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