Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: scenemanager clean up ScenesLoaded after synch unloading of remaining scenes not used [back port] #2977

Conversation

NoelStephensUnity
Copy link
Collaborator

Fix for the NetworkManager.ScenesLoaded not being updated when PostSynchronizationSceneUnloading is set to true and there are scenes not used for a synchronization.

This is a back-port of #2971.

Changelog

  • Fixed: Issue where NetworkManager.ScenesLoaded was not being updated if PostSynchronizationSceneUnloading was set and any loaded scenes not used during synchronization were unloaded.

Testing and Documentation

  • Includes integration test update to NetworkSceneManagerSceneVerification.SceneVerifyBeforeLoadTest.
  • No documentation changes or additions were necessary.

remove the scene from the scenesLoaded dictionary when there are remaining scenes to be unloaded.
back ported test updates
Adding change log entry
@NoelStephensUnity NoelStephensUnity marked this pull request as ready for review July 18, 2024 15:57
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner July 18, 2024 15:57
Copy link
Collaborator

@fluong6 fluong6 left a comment

Choose a reason for hiding this comment

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

LGTM

@NoelStephensUnity NoelStephensUnity merged commit f0db817 into develop Jul 18, 2024
24 checks passed
@NoelStephensUnity NoelStephensUnity deleted the fix/scenemanager-unload-scenes-after-synch-clean-up-back-port-1-x branch July 18, 2024 19:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants