Skip to content

Conversation

@docEdub
Copy link
Contributor

@docEdub docEdub commented Nov 21, 2025

With the new playground the monaco-editor package has errors due to missing sourcemaps. This change takes the upstream fix for this and updates our code to fix the deprecation issues introduced by the update.

Note that the webpack monacoEditorPath option has to be set now to avoid new webpack errors occurring when building/running the playground.

With the new playground the monaco-editor package has errors due to missing sourcemaps. This change takes the upstream fix for this and updates our code to fix the deprecation issues introduced by the update.

Note that the webpack `monacoEditorPath` option has to be set now to avoid new webpack errors occurring when building/running the playground.
@docEdub docEdub enabled auto-merge (squash) November 21, 2025 18:34
@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 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 Nov 21, 2025

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

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 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/17493/merge/

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 2025

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

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

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

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/17493/merge/?snapshot=refs/pull/17493/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 Nov 21, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 2025

@sebavan sebavan marked this pull request as draft November 21, 2025 23:24
@sebavan
Copy link
Member

sebavan commented Nov 21, 2025

Moving to draft to prevent over excitement merge before the weekend :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants