chore: Add DA ConnectionApprovalCallback warnings #3658
Merged
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.
Purpose of this PR
A post on the discussions board pointed out that we don't have any information around the
ConnectionApprovalCallback
indicating that it won't work with distributed authority.This PR:
StartClient
?)Jira ticket
n/a
Changelog
NetworkManager.NetworkConfig.ConnectionApproval
is set in a DA context.Documentation
Testing & QA (How your changes can be verified during release Playtest)
Mostly a docs update, no manual testing is needed.
Functional Testing
Manual testing :
Manual testing done
Automated tests:
Covered by existing automated tests
Covered by new automated tests
Does the change require QA team to:
Review automated tests
?Execute manual tests
?Provide feedback about the PR
?If any boxes above are checked the QA team will be automatically added as a PR reviewer.
Backports
Only a DA issue so 2.x only.
Docs screenshots
Connection Approval:
NetworkConfig settings
NetworkManager start