Skip to content

Ensure react-dom/client is built in Codesandbox preview builds - #30757

Merged
eps1lon merged 1 commit into
react:mainfrom
eps1lon:fix-csb
Aug 20, 2024
Merged

eps1lon merged 1 commit into
react:mainfrom
eps1lon:fix-csb

Conversation

@eps1lon

@eps1lon eps1lon commented Aug 20, 2024 •

Copy link
Copy Markdown
Collaborator

Summary

react-dom/client wasn't usable in Codesandbox preview builds because we never built it. This used to work because the client APIs used to be exported from react-dom/index but now aren't so we need to explicitly list the entrypoint.

How did you test this change?

@vercel

vercel Bot commented Aug 20, 2024 •

Copy link
Copy Markdown

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

Name Status Preview Comments Updated (UTC)
react-compiler-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 20, 2024 8:00pm

@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Aug 20, 2024
@react-sizebot

Copy link
Copy Markdown

Comparing: 92d26c8e93a88ca41338d3509b4324ad19a89c1e...36dd945f195047618d4f4d7f16ad7f2756fb46de

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.68 kB 6.68 kB +0.05% 1.82 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 500.37 kB 500.37 kB = 89.80 kB 89.80 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB +0.05% 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 507.50 kB 507.50 kB = 90.96 kB 90.96 kB
facebook-www/ReactDOM-prod.classic.js = 595.24 kB 595.24 kB = 105.55 kB 105.55 kB
facebook-www/ReactDOM-prod.modern.js = 571.54 kB 571.54 kB = 101.75 kB 101.75 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 3aee977

@eps1lon
eps1lon merged commit 4c2dfb3 into react:main Aug 20, 2024
@eps1lon
eps1lon deleted the fix-csb branch August 20, 2024 20:12
eps1lon added a commit to vercel/next.js that referenced this pull request Aug 23, 2024

This branch was successfully deployed

1 active deployment
Preview — 3aee977c Deployed Aug 20, 2024 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants