Skip to content

fix: Enable custom messages to be sent to self when running as host. #2296

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

Merged
merged 6 commits into from
Nov 9, 2022

Conversation

ShadauxCat
Copy link
Collaborator

Changelog

  • Fixed: Custom messages are now properly received by the local client when they're sent while running in host mode.

Testing and Documentation

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

@ShadauxCat ShadauxCat requested a review from a team as a code owner November 3, 2022 21:28
@@ -18,6 +18,7 @@ Additional documentation and release notes are available at [Multiplayer Documen
- Fixed issue where in-scene placed `NetworkObjects` were not honoring the `AutoObjectParentSync` property. (#2281)
- Fixed the issue where `NetworkManager.OnClientConnectedCallback` was being invoked before in-scene placed `NetworkObject`s had been spawned when starting `NetworkManager` as a host. (#2277)
- Creating a `FastBufferReader` with `Allocator.None` will not result in extra memory being allocated for the buffer (since it's owned externally in that scenario). (#2265)
- Custom messages are now properly received by the local client when they're sent while running in host mode. (#2296)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor nit-pick:
Could you move that to the top of the fixed items to maintain the descending PR number order?

Copy link
Collaborator

@NoelStephensUnity NoelStephensUnity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the minor nit-pick on the changelog, it all looks good to me!

@ShadauxCat ShadauxCat enabled auto-merge (squash) November 8, 2022 17:34
@ShadauxCat ShadauxCat merged commit 8fb7c61 into develop Nov 9, 2022
@ShadauxCat ShadauxCat deleted the fix/custom_messages_send_to_self branch November 9, 2022 17:56
jakobbbb pushed a commit to GooseGirlGames/com.unity.netcode.gameobjects that referenced this pull request Feb 22, 2023
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.

3 participants