Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WARNING: Please make your PR against the
develop
branch.Select the type of change:
Purpose of the Pull Request:
I finished this quickstart guide recently but ran into some issues throughout the tutorial. I wanted to address updates to this page to better help the flow of going through this part of the docs. Here are the details of what I changed:
HelloWorldManager.cs
example with the following:NetworkManager
is now static, as it caused build errors with other static methods and probably should be its own instance anywaySubmitNewPosition()
sinceHelloWorldPlayer.cs
is not introduced until later in the tutorial, but before building for other stepsTest the RPCs
section, I removed the build steps and replaced it with setting up Multiplayer Play Mode. You can't see logs through builds unless you enable development builds, and even then onlyDebug.LogError
is displayedNetworkVariable
position inHelloWorldPlayer.cs
anymore since technicallyNetworkTransform
handles this out of boxAs a note, with Unity 6 out some of the general for creating scripts/running builds are slightly different now. I held off on these changes since they were small, but wanted to bring this up for awareness.
Issue Number: N/A