fix: Distributed authority quickstart #1458
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jabbacakes
approved these changes
Apr 4, 2025
| @@ -31,6 +31,10 @@ You also need the following packages and minimum versions for a WebGL distribute | |||
|
|
|||
| Follow the instructions on the [distributed authority general quickstart](distributed-authority-quick-start.md) page. | |||
|
|
|||
| ## Configure the transport to use websockets | |||
|
|
|||
| Enable the "Use Web Sockets" checkbox on the **DistributedAuthorityTransport** component that was added to the *NetworkManager* object.  | |||
There was a problem hiding this comment.
Suggested change
| Enable the "Use Web Sockets" checkbox on the **DistributedAuthorityTransport** component that was added to the *NetworkManager* object.  | |
| Enable the **Use Web Sockets** checkbox on the **DistributedAuthorityTransport** component that was added to the *NetworkManager* object.  |
UI elements/names should be in bold formatting
There was a problem hiding this comment.
Thanks! I copied the formatting out of other places in the DA quickstart so I've gone through and updated all the formatting in these two files so I don't confuse myself in the future 😄
There was a problem hiding this comment.
Consistency, in this economy?! (thanks tho; I've been meaning to do a pass for it throughout the whole docs, but it's just never been a high enough priority)
There was a problem hiding this comment.
Gotta save future Emma
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Select the type of change:
Purpose of the Pull Request:
DistributedAuthorityTransport.Note:
The sessions SDK has a flow where if the
DistributedAuthorityTransportdoes not exist, then it will create the transport, however it will not copy over any configured variables for any other transport that may exist. This means if you follow the current instructions to add aUnityTransportand have any custom configuration then the user will face issues.Issue Number:
MTTB-1113
MTT-11876