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(react-email): Proper CJS support #1786

Open
wants to merge 9 commits into
base: canary
Choose a base branch
from

Conversation

gabrielmfern
Copy link
Collaborator

This does a small change to the rendering utilities exporter esbuild plugin
so that it maintains the original value of module.exports in a way that
we can both treat default exports for ESM, i.e. module.exports.default,
and treat the default export for CJS as well.

Another small thing this PR does is fix a potential file system race condition
on the code that checks if a file is an email template by using file descriptors.

@gabrielmfern gabrielmfern added App: Preview Package: react-email This is the CLI we generally use as just `email` on the temrinal. labels Nov 21, 2024
@gabrielmfern gabrielmfern self-assigned this Nov 21, 2024
Copy link

changeset-bot bot commented Nov 21, 2024

🦋 Changeset detected

Latest commit: 08b58f6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
react-email Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Nov 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-email ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 21, 2024 3:19pm
react-email-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 21, 2024 3:19pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App: Preview Package: react-email This is the CLI we generally use as just `email` on the temrinal.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant