-
Notifications
You must be signed in to change notification settings - Fork 9
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
Notify workflow failure with Email when the workflow is triggered by webhooks #333
Notify workflow failure with Email when the workflow is triggered by webhooks #333
Conversation
This method would be called from webhoook.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…re-triggered-by-webhook
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
bun.lockb
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nits: @satococoa
I would appreciate it if you could delete this file!
ref: #306
Summary
This Pull Request introduces the following improvements and features:
Changes
sendEmail
function using nodemailer, providing configurable SMTP settings through environment variables.SEND_EMAIL_DEBUG
)onStepFail
callbackreportAgentTimeUsage
agentId
explicitly and querying the appropriate team and subscription details.Why nodemailer?
nodemailer vs. emailjs
https://npmtrends.com/emailjs-vs-nodemailer
nodemailer vs. aws-sdk/client-ses
TODO
SMTP_XXXX
environment values to PreviewSMTP_XXXX
environment values to Production