-
-
Notifications
You must be signed in to change notification settings - Fork 65
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
WIP: merge clientscene and networkclient #84
Conversation
@@ -152,7 +152,7 @@ void SendAnimationMessage(int stateHash, float normalizedTime, int layerId, byte | |||
{ | |||
RpcOnAnimationClientMessage(stateHash, normalizedTime, layerId, parameters); | |||
} | |||
else if (ClientScene.readyConnection != null) | |||
else if (client.connection != null) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is so much better!
We could even merge this without the rest of the stuff.
This is not a problem in UPM
We already have NetworkClient.hostServer for this exact purpose
unspawning an object is now consistent between DestroyObject and DestroyAllObjects by refactoring into Unspawn method
Here is an overview of what got changed by this pull request: Issues
======
+ Solved 1
- Added 5
Complexity increasing per file
==============================
- Assets/Mirror/Runtime/NetworkClient.cs 3
See the complete overview on Codacy |
Kudos, SonarCloud Quality Gate passed! 0 Bugs |
# [20.0.0](19.1.2-master...20.0.0-master) (2020-03-20) ### Bug Fixes * compilation issue after merge from mirror ([daf07be](daf07be)) * merge clientscene and networkclient (#84) ([dee1046](dee1046)), closes [#84](#84) ### Features * SyncSet and SyncDictionary now show in inspector ([#1561](https://github.com/MirrorNG/MirrorNG/issues/1561)) ([5510711](5510711)) ### BREAKING CHANGES * ClientScene is gone
Merge clientScene and NetworkClient, getting rid of all that static state. BREAKING CHANGE: ClientScene is gone
# [20.0.0](19.1.2-master...20.0.0-master) (2020-03-20) ### Bug Fixes * compilation issue after merge from mirror ([daf07be](daf07be)) * merge clientscene and networkclient (#84) ([dee1046](dee1046)), closes [#84](#84) ### Features * SyncSet and SyncDictionary now show in inspector ([#1561](https://github.com/MirrorNG/MirrorNG/issues/1561)) ([5510711](5510711)) ### BREAKING CHANGES * ClientScene is gone
🎉 This PR is included in version 20.0.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
No description provided.