Skip to content

Conversation

0xFA11
Copy link
Contributor

@0xFA11 0xFA11 commented May 17, 2021

NetworkBehaviour.NetworkObject property no longer throws an exception:

  • NetworkBehaviour.NetworkObject property getter logs a network warning and returns null instead.
  • NetworkBehaviour.HasNetworkObject is now relying on NetworkBehaviour.NetworkObject directly.
  • NetworkBehaviourTests are also updated accordingly.

@0xFA11 0xFA11 enabled auto-merge (squash) May 17, 2021 18:55
Copy link
Contributor

@TwoTenPvP TwoTenPvP left a comment

Choose a reason for hiding this comment

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

👌🏻

@0xFA11 0xFA11 force-pushed the refactor/netbhv-netobj-prop branch from d743c4b to 3de5a05 Compare May 18, 2021 12:06
@0xFA11 0xFA11 merged commit 2774a2c into develop May 18, 2021
@0xFA11 0xFA11 deleted the refactor/netbhv-netobj-prop branch May 18, 2021 15:20
SamuelBellomo added a commit that referenced this pull request May 19, 2021
* develop:
  fix: gracefully handle exceptions in an RPC invoke (#846)
  fix: Fixing utp license file for legal (#843)
  ci: enable standards check on UTP too (#837)
  refactor: NetworkBehaviour.NetworkObject no longer throws an exception (#838)
  fix: revert #830 (#840)
  test: converting the manual rpc tests over to an automated unit test (#830)
SamuelBellomo added a commit that referenced this pull request May 21, 2021
…player-object-get

* develop:
  test: manual connection approval to fully automated connection approval testing (#839)
  test: manual rpc tests to automated fixed revision (#841)
  fix: CleanDiffedSceneObjects was not clearing PendingSoftSyncObjects  (#834)
  feat: NetworkTransform now uses NetworkVariables instead of RPCs (#826)
  fix: gracefully handle exceptions in an RPC invoke (#846)
  fix: Fixing utp license file for legal (#843)
  ci: enable standards check on UTP too (#837)
  refactor: NetworkBehaviour.NetworkObject no longer throws an exception (#838)
  fix: revert #830 (#840)
  test: converting the manual rpc tests over to an automated unit test (#830)

# Conflicts:
#	com.unity.multiplayer.mlapi/Tests/Runtime/MultiInstance/MultiInstanceHelpers.cs
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.

2 participants