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

fix: sdk hot reload models caching #2095

Merged
merged 1 commit into from
Sep 19, 2024
Merged

Conversation

pravusjif
Copy link
Member

@pravusjif pravusjif commented Sep 18, 2024

WHY

During local scene development, when using the hot reload (scene reloads when any change in the sdk-side of the locally running scene is made) the 3D models are not updated whtn the scene is reloaded. So if the creator had changed the model, after the hot reload they still see the old verion.

Issue: #2082

WHAT

Added local scene development check when using cache at gltfAsset preparation.

DEMO VIDEO WITH FIX

Screen.Recording.2024-09-18.at.6.02.00.PM.mp4

@pravusjif pravusjif added no QA needed Used to tag pull requests that does not require QA validation sdk labels Sep 18, 2024
@pravusjif pravusjif self-assigned this Sep 18, 2024
Copy link
Contributor

github-actions bot commented Sep 18, 2024

badge

Windows and Mac build successfull in Unity Cloud! You can find a link to the downloadable artifact below.

Name Link
Commit 922dac8
Logs https://github.com/decentraland/unity-explorer/actions/runs/10925988973
Download Windows https://github.com/decentraland/unity-explorer/suites/28551205524/artifacts/1949098920
Download Mac https://github.com/decentraland/unity-explorer/suites/28551205524/artifacts/1949332366
Built on 2024-09-18T18:03:57Z

@pravusjif pravusjif marked this pull request as ready for review September 18, 2024 16:08
Copy link
Collaborator

Choose a reason for hiding this comment

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

👏

@pravusjif pravusjif merged commit 0edde11 into main Sep 19, 2024
10 of 11 checks passed
@pravusjif pravusjif deleted the fix/sdk-hot-reload-models-caching branch September 19, 2024 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no QA needed Used to tag pull requests that does not require QA validation sdk
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants