Skip to content

fix: NetworkList to call INetworkSerializable's NetworkSerialize() method properly #1682

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 5 commits into from
Feb 18, 2022

Conversation

ShadauxCat
Copy link
Collaborator

MTT-1687
#1396

Changelog

com.unity.netcode.gameobjects

  • Fixed: NetworkList now properly calls INetworkSerializable's NetworkSerialize() method

Testing and Documentation

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

@ShadauxCat ShadauxCat changed the title fix: NetworkList noew properly calls INetworkSerializable's NetworkSe… fix: NetworkList noew properly calls INetworkSerializable's NetworkSerialize() method Feb 11, 2022
@ShadauxCat
Copy link
Collaborator Author

This PR can't be merged until 59e8874 is backported into develop, as it depends on code from that commit.


### Fixed

- Fixed: NetworkList now properly calls INetworkSerializable's NetworkSerialize() method (#1682)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Fixed: NetworkList now properly calls INetworkSerializable's NetworkSerialize() method (#1682)
- Fixed NetworkList to call INetworkSerializable's NetworkSerialize() method properly (#1682)

@0xFA11 0xFA11 changed the title fix: NetworkList noew properly calls INetworkSerializable's NetworkSerialize() method fix: NetworkList to call INetworkSerializable's NetworkSerialize() method properly Feb 14, 2022
@0xFA11
Copy link
Contributor

0xFA11 commented Feb 17, 2022

#1487 for tracking

@0xFA11
Copy link
Contributor

0xFA11 commented Feb 17, 2022

apparently PR #1653 got the dependent #1487 backported already.

@0xFA11 0xFA11 enabled auto-merge (squash) February 17, 2022 22:58
@0xFA11 0xFA11 disabled auto-merge February 17, 2022 23:19
@0xFA11 0xFA11 force-pushed the fix/INetworkSerializable_in_NetworkList_develop branch from 26b4f35 to 0af4957 Compare February 18, 2022 04:35
@0xFA11 0xFA11 force-pushed the fix/INetworkSerializable_in_NetworkList_develop branch from 4a9caa0 to b8f1de8 Compare February 18, 2022 04:51
@0xFA11 0xFA11 enabled auto-merge (squash) February 18, 2022 04:51
@0xFA11 0xFA11 merged commit 302b0b2 into develop Feb 18, 2022
@0xFA11 0xFA11 deleted the fix/INetworkSerializable_in_NetworkList_develop branch February 18, 2022 05:22
WhippetsAintDogs added a commit to fikaproductions/multiplayer-community-contributions that referenced this pull request Apr 12, 2022
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