-
Notifications
You must be signed in to change notification settings - Fork 811
fix(react-email): Missing Headers on the global context for email templates #1446
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
fix(react-email): Missing Headers on the global context for email templates #1446
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@ugurrdemirel is attempting to deploy a commit to the resend Team on Vercel. A member of the Team first needs to authorize it. |
Can you also format? |
packages/code-block/src/prism.ts
Outdated
@@ -7876,8 +7876,8 @@ export { Prism }; | |||
return "string" == typeof e |
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.
It seems to be failing formatting check due to the changes here, were they done by prettier? IIRC this file should be ignored.
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.
Hello, prettier made changes at this file when I run format
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.
Weirdly, if I do format on my machine, it changes this file back to what it was before. It might be a Unix-Windows thing, not sure, adding it to a .prettierignore
either way.
Hello, I noticed
Headers
doesn't in theget-email-component.ts
. Because of this, you can't see the preview of your email template if you are usingHeaders
in your code. This PR should fix it.(BTW I spent 4 hours while investigating why tf is this can't find Headers 🤣)