Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add react email implementation #574

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

sdrejkarz
Copy link
Contributor

@sdrejkarz sdrejkarz commented May 22, 2024

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes/features)
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce?

Add React.email implementation and remove styled-components.
After the merge this issue is redundant.

What is the current behavior?

Emails are styled by styled-components - issue

What is the new behavior?

Emails are styled by tailwind

screen shot from react.email platform

@sdrejkarz sdrejkarz force-pushed the feat/react-email-implementation branch from 47c5c0e to f73e2bb Compare May 22, 2024 13:45
@sdrejkarz sdrejkarz marked this pull request as ready for review May 22, 2024 14:18
@sdrejkarz sdrejkarz force-pushed the feat/react-email-implementation branch from f73e2bb to 2e97336 Compare May 23, 2024 11:50
@@ -29,7 +29,7 @@ emails at the most optimal time for their users.

### Sending test emails

Sending test emails is made easy with the ability to send test emails directly from a Storybook to Mailcatcher service.
Sending test emails is made easy with the ability to send test emails directly from a React.Email via Resend.
Copy link
Contributor Author

@sdrejkarz sdrejkarz May 23, 2024

Choose a reason for hiding this comment

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

Built-in Resend works fine, but I haven't managed to send the image (logo).

"@sb/webapp-api-client": "workspace:*",
"@sb/webapp-core": "workspace:*",
"@sb/webapp-finances": "workspace:*",
"juice": "^8.1.0"
"juice": "^8.1.0",
"react-email": "2.1.3"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

niticking: I'm thinking of moving react-email to the devDependencies, as it will be used only locally.

@sdrejkarz sdrejkarz force-pushed the feat/react-email-implementation branch from 3298c40 to 446f538 Compare May 27, 2024 08:02
@sdrejkarz sdrejkarz force-pushed the feat/react-email-implementation branch from 446f538 to b377549 Compare May 27, 2024 08:02
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.

1 participant