Skip to content

Add React fast refresh to Sandbox and Playground #16860

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ryantrem
Copy link
Member

@ryantrem ryantrem commented Jul 8, 2025

This change enables React fast refresh in both Sandbox and Playground. This means when changes are made to React components in .tsx files (as long as the React components are exported), the changes will be reflected "immediately" (within about 5 seconds or so).

@ryantrem ryantrem requested a review from sebavan July 8, 2025 21:25
@ryantrem ryantrem enabled auto-merge (squash) July 8, 2025 21:25
@bjsplat
Copy link
Collaborator

bjsplat commented Jul 8, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 8, 2025

Reviewer - this PR has made changes to one or more package.json files.

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 8, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 8, 2025

You have made possible changes to the playground.
You can test the snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16860/merge/

The snapshot playground with the CDN snapshot (only when available):

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16860/merge/?snapshot=refs/pull/16860/merge

Note that neither Babylon scenes nor textures are uploaded to the snapshot directory, so some playgrounds won't work correctly.

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 8, 2025

Building or testing the sandbox has failed.

If the tests failed, results can be found here:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16860/merge/testResults/

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 8, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 8, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 8, 2025

Building or testing the sandbox has failed.

If the tests failed, results can be found here:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16860/merge/testResults/

1 similar comment
@bjsplat
Copy link
Collaborator

bjsplat commented Jul 8, 2025

Building or testing the sandbox has failed.

If the tests failed, results can be found here:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16860/merge/testResults/

Copy link
Member

@sebavan sebavan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the config be added to the webpacktools ?

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 9, 2025

Building or testing the sandbox has failed.

If the tests failed, results can be found here:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16860/merge/testResults/

@sebavan
Copy link
Member

sebavan commented Jul 14, 2025

Any update on this one ? mostly curious about why the tools are failing to build

@deltakosh
Copy link
Contributor

Up @ryantrem :)

@ryantrem
Copy link
Member Author

Just getting back to this now. I moved the configuration logic into the shared tooling (as best I could with the current structure) and updated all three projects to just set the flags for this config logic.

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

Reviewer - this PR has made changes to one or more package.json files.

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

You have made possible changes to the playground.
You can test the snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16860/merge/

The snapshot playground with the CDN snapshot (only when available):

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16860/merge/?snapshot=refs/pull/16860/merge

Note that neither Babylon scenes nor textures are uploaded to the snapshot directory, so some playgrounds won't work correctly.

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

Building or testing the sandbox has failed.

If the tests failed, results can be found here:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16860/merge/testResults/

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

Reviewer - this PR has made changes to one or more package.json files.

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

You have made possible changes to the playground.
You can test the snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16860/merge/

The snapshot playground with the CDN snapshot (only when available):

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16860/merge/?snapshot=refs/pull/16860/merge

Note that neither Babylon scenes nor textures are uploaded to the snapshot directory, so some playgrounds won't work correctly.

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

You have changed file(s) that made possible changes to the sandbox.
You can test the sandbox snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16860/merge/

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jul 14, 2025

@ryantrem ryantrem requested a review from sebavan July 14, 2025 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants