-
Notifications
You must be signed in to change notification settings - Fork 363
[Resend] Update the resend component #796
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
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
hmm i'm still getting
Uncaught Error: Resend API error: {"statusCode":403,"message":"The domains are not verified. Please, add and verify your domains on https://resend.com/domains","name":"validation_error"}
This is what I was saying I was running into that made me think the resend domain didn't "just work"
usingOpenAi: modelProvider == 'OpenAI', | ||
usingGoogle: modelProvider == 'Google', | ||
resendProxyEnabled: getEnv('RESEND_PROXY_ENABLED') == '1', | ||
resendProxyEnabled: !featureFlags.enableResend && getEnv('RESEND_PROXY_ENABLED') == '1', |
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.
good idea
did anything show up in your email? |
no, here's a share link https://chef-git-jordan-cleanup-resend-component.previews.convex.dev/share/e512b1 |
@ikhare the resend domain doesn't work if you already have a verified domain in your account (I do for a personal project). So when I use the resend component with chef, it fails. we could branch on whether you are new to resend or if you already have a verified domain - the AI agent can ask the user and from there prompt for the second environment variable for your domain I think we should do something about this, open to ideas, because it's also a footgun if you do add a registered domain it stops working and it's not at all obvious via the UI generated in the app. maybe that's a separate problem with the UI though. |
Cleans up the resend component docs by:
RESEND_DOMAIN
from setup