Skip to content

Add second warning email template for delete inactive accounts#18204

Closed
vbudhram wants to merge 2 commits intomainfrom
fxa-10637
Closed

Add second warning email template for delete inactive accounts#18204
vbudhram wants to merge 2 commits intomainfrom
fxa-10637

Conversation

@vbudhram
Copy link
Copy Markdown
Contributor

Because

  • We want to give users another notice before we delete the account

This pull request

  • Adds the new email template

Issue that this pull request solves

Closes: https://mozilla-hub.atlassian.net/browse/FXA-10637

Checklist

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

Other information (Optional)

This is based off #18202 and has a lot of duplicate code. This is fine since it gives a clear 1:1 mapping of emails and will be easier to change. Blocked until that PR is merged.

vpomerleau and others added 2 commits January 8, 2025 17:40
Because:

* We will be sending out warnings before deleting inactive accounts

This commit:

* Adds new email template for inactiveAccountFirstWarning including l10n and storybook
* Adds new partials for use in inactive account emails
* Adds stories for automated email partials

Closes #FXA-10636
@vbudhram vbudhram self-assigned this Jan 10, 2025
@vbudhram vbudhram marked this pull request as ready for review January 13, 2025 15:43
@vbudhram vbudhram requested review from a team as code owners January 13, 2025 15:43
@bcolsson
Copy link
Copy Markdown
Contributor

This PR has stale strings that were changed in #18202

@vbudhram
Copy link
Copy Markdown
Contributor Author

@bcolsson Ah, looks like I need to rebase this.

@vpomerleau
Copy link
Copy Markdown
Contributor

Ah, looks like I need to rebase this.

Ping me when rebased and I'll review 😄

@vbudhram
Copy link
Copy Markdown
Contributor Author

@vpomerleau @bcolsson Closing this in favor of #18218

@vbudhram vbudhram closed this Jan 14, 2025
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