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.

Sign up for GitHub

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

Jump to bottom

[TRI-951] Remove resend dependency from the webapp and replace with nodemailer and smtp #282

Open
ericallam opened this issue Aug 8, 2023 · 2 comments · May be fixed by #313
Open

[TRI-951] Remove resend dependency from the webapp and replace with nodemailer and smtp #282

ericallam opened this issue Aug 8, 2023 · 2 comments · May be fixed by #313
Labels
area/platforms enhancement New feature or request

Comments

@ericallam
Copy link
Member

ericallam commented Aug 8, 2023 •
edited

Now that Resend.com supports sending email via SMTP, we should convert the webapp to use a native SMTP mailer instead of the Resend SDK. We will need to replace the RESEND_API_KEY with SMTP_* environment variables (which should still be optional). When SMTP_* env vars are missing, we should continue to print the Magic Link login emails to the console how we are currently doing.

This will allow self-hosters to send email* without having to sign up to Resend.com and possibly use an existing email provider.

  • Currently only used for Magic Link login

TRI-951
@maige-app maige-app bot added enhancement New feature or request area/platforms labels Aug 8, 2023
@ericallam ericallam changed the title Remove resend dependency from the webapp and replace with nodemailer and smtp [TRI-951] Remove resend dependency from the webapp and replace with nodemailer and smtp Aug 8, 2023
@Chigala
Copy link
Contributor

Chigala commented Aug 11, 2023

I'm taking this up

@Chigala Chigala linked a pull request Aug 11, 2023 that will close this issue
Open
@Flowko
Copy link

Flowko commented Mar 10, 2024

any news on this ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/platforms enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

added SMTP support and removed resend dependency Chigala/trigger.dev
3 participants
@ericallam @Flowko @Chigala