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

Make sure to define a _jsxFilename for custom components #1306

Merged
merged 2 commits into from Nov 8, 2022

Conversation

Janpot
Copy link
Member

@Janpot Janpot commented Nov 8, 2022

This came up in #1294

It looks like Next.js 13 compiles away the global _jsxFileName variable in production (the new SWC minifier I presume?). Custom components were implicitly relying on this variable existing. This PR brings it back explicitly.

@render
Copy link

render bot commented Nov 8, 2022

@Janpot Janpot added the core Infrastructure work going on behind the scenes label Nov 8, 2022
@Janpot Janpot marked this pull request as ready for review November 8, 2022 09:38
@Janpot Janpot mentioned this pull request Nov 8, 2022
10 tasks
@Janpot Janpot enabled auto-merge (squash) November 8, 2022 09:55
@Janpot Janpot merged commit ffba527 into master Nov 8, 2022
@Janpot Janpot deleted the supply-jsxfilename branch November 8, 2022 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Infrastructure work going on behind the scenes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants