Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix: NetworkObject.SpawnWithObservers was not being honored for late joining clients [MTT-6934] #2623

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

Merged

Conversation

NoelStephensUnity
Copy link
Collaborator

@NoelStephensUnity NoelStephensUnity commented Jul 12, 2023

This resolves the issue where NetworkObject.SpawnWithObservers was not being honored for late joining clients.

MTT-6934
fix: #2609

Changelog

  • Fixed: Issue where NetworkObject.SpawnWithObservers was not being honored for late joining clients.

Testing and Documentation

  • Includes integration test modifications to NetworkObjectOnSpawnTests.
  • No documentation changes or additions were necessary.

This resolves the issue where NetworkObjects with SpawnWithObservers set to false was not being honored for late joining clients.
This also resolves an issue when scene management is disabled the NetworkObjects' observers were not being updated for newly connected clients.
This validates the fix for the late joining client observer issue.
adding change log entry
whitespace fix
@NoelStephensUnity NoelStephensUnity marked this pull request as ready for review July 12, 2023 20:19
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner July 12, 2023 20:19
- Fixed a failing UTP test that was failing when you install Unity Transport package 2.0.0 or newer.

- Fixed issue where `NetworkObject.SpawnWithObservers` was not being honored for late joining clients. (#2623)
- Fixed a failing UTP test that was failing when you install Unity Transport package 2.0.0 or newer. (#2616)
Copy link
Collaborator

Choose a reason for hiding this comment

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

"Not your change", heh... but...

I asked Frank to remove this changelog in the release PR, can we remove it here too since you're touching it anyway? I don't generally think we need to broadcast test changes.

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.

Object visibility with late joining players - SpawnWithObservers & CheckObjectVisibility
2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.