Skip to content

Create Email Templates - #156

Merged
Sectonic merged 5 commits into
mainfrom
Ben/152-Email-Templates
Sep 15, 2025
Merged

Sectonic merged 5 commits into
mainfrom
Ben/152-Email-Templates

Conversation

@Blance5

@Blance5 Blance5 commented Sep 7, 2025

Copy link
Copy Markdown
Contributor

Description

Created the 5 email templates described in issue #152.

Checklist

  • [x ] The ticket is mentioned above
  • [x ] The changes fulfill the success criteria of the ticket
  • [x ] The changes were self-reviewed
  • A review was requested
image image image image image

@netlify

netlify Bot commented Sep 7, 2025 •

Copy link
Copy Markdown

❌ Deploy Preview for hope-for-haiti failed. Why did it fail? →

Name Link
🔨 Latest commit 96e76b3
🔍 Latest deploy log https://app.netlify.com/projects/hope-for-haiti/deploys/68c79f10b78e300008b63842

@Sectonic
Sectonic self-requested a review September 7, 2025 23:27

@Sectonic Sectonic 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.

Use tailwind instead of JSS. This is to keep consistent with the rest of the app, take a look at https://react.email/docs/components/tailwind. The other LT members will take a look at the designs and give further feedback on them.

@Sectonic

Sectonic commented Sep 8, 2025

Copy link
Copy Markdown
Contributor

You can import the config from the exported tailwind.config.ts probably

@Sectonic
Sectonic marked this pull request as draft September 8, 2025 01:28
@Sectonic Sectonic linked an issue Sep 8, 2025 that may be closed by this pull request

@Sectonic Sectonic 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.

Great job switching to tailwind.

For the CreatePartnerAccountInvite and CreatePartnerAccountReminder templates, change the names of the files and components to be just CreateAccountInvite and CreateAccountReminder because we will be sending those to new internal and administrative users as well.

For some reason the fonts of the create account ones and the other ones are different. The other ones should have the same font as the create account invite and reminder templates.

Also do not say "feel free to reply to this email" because that is not how our emailing system will work. In each email say to email the contact person (yvette@hopeforhaiti.com)

@Sectonic

Copy link
Copy Markdown
Contributor

Putting the new designs of the emails here just for reference.
image
image
image
image
image

@Sectonic
Sectonic merged commit 5390ec8 into main Sep 15, 2025
0 of 4 checks passed
@Sectonic
Sectonic deleted the Ben/152-Email-Templates branch September 15, 2025 05:19
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.

Email Templates

2 participants