Skip to content

fix: player spawn not applying position and rotation via connection approval #3078

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

Conversation

NoelStephensUnity
Copy link
Collaborator

@NoelStephensUnity NoelStephensUnity commented Sep 26, 2024

Fixing regression bug where applying the position and/or rotation to the NetworkManager.ConnectionApprovalResponse when connection approval and auto-spawn player prefab are enabled would not apply the position and/or rotation when the player prefab was instantiated.

MTTB-481

fix: #3074

Changelog

  • Fixed: Issue where applying the position and/or rotation to the NetworkManager.ConnectionApprovalResponse when connection approval and auto-spawn player prefab were enabled would not apply the position and/or rotation when the player prefab was instantiated.

Testing and Documentation

  • Includes integration tests updates to ConnectionApprovalTests.
  • No documentation changes or additions were necessary.

Apply the position and rotation set by the NetworkManager.ConnectionApprovalResponse when connection approval is enabled and auto-spawn player prefabs is enabled.
Validation test for this PR
adding changelog entry
@NoelStephensUnity NoelStephensUnity marked this pull request as ready for review September 26, 2024 23:00
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner September 26, 2024 23:00
@NoelStephensUnity NoelStephensUnity merged commit 390c332 into develop-2.0.0 Sep 27, 2024
24 checks passed
@NoelStephensUnity NoelStephensUnity deleted the fix/player-spawn-not-applying-position-rotation-via-connection-approval branch September 27, 2024 12:14
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