-
Notifications
You must be signed in to change notification settings - Fork 7.8k
Fix Activity sandboxes #8035
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 Activity sandboxes #8035
Conversation
Size changes📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
well, that doesn't work |
This is a total hack, it looks like sandpack is resolving to the wrong versions. Like if you put import { unstable_Activity, Activity as ActivityStable} from 'react';
let Activity = ActivityStable ?? unstable_Activity; Until we figure out how to fix sandpack. |
I'm looking at this now. Will try to repro it on my side and see what we need to fix to resolve 19.2 correctly! |
* Fix Activity sandboxes * hacky fix
* Use stable release in demos with HTML output * Fix Activity sandboxes (#8035) * Fix Activity sandboxes * hacky fix * Revert "Fix Activity sandboxes (#8035)" This reverts commit f9fd07c. * Bump to 19.2 * cleanup rest of sandpacks * Pin deps * Add back package.json with third-party deps * "^" should be fixed now * add back package.json for more toastify deps * fuggin whitespace * more fuggin whitespace * fml like fr fr --------- Co-authored-by: Sebastian Sebbie Silbermann <sebastian.silbermann@vercel.com> Co-authored-by: Ricky <rickhanlonii@gmail.com> Co-authored-by: Rick Hanlon <rickhanlonii@meta.com>
Hopefully fixes #8034