Bannerlord-Coop-Team / BannerlordCoop

MIT License
256 stars 50 forks source link

Sync VillagerPartyComponent #800

Open garrettluskey opened 5 months ago

garrettluskey commented 5 months ago

Description

  1. Sync the values in the Synced list below.
  2. Create an integration test for all values (1 for server changing the value and 1 for client changing the value if applicable)
  3. Create a command to test changing the value in game see similar command

Also see additional information at the bottom for more information and templates

Key Server side only (only allow the running of the function on the server side) Client side only (only allow the running of the function on the client side) Client side sync (request the server to change the value, server then changes the value and replies to the client allowing the change) Server side sync (only server allows running of the function and send to all clients the value changed and clients update the value/call original function)

Synced

Method/Field Name Sync Type
Village Server side sync

Deferred

None

Externally Deferred

None

Non-Synced None

Intended Design

Message networking image Integration Testing image

Location

Create a branch based from development

Related Issues

N/A

Requirements

N/A

Additional information

Using the GameInterface Service Tempale create commands to test the synced values.

For network communication use Server Service Template and Client Service Template

It is recommended to start with an integration test, you can create one using the test template.

Definition of Done

dennispost99 commented 4 months ago

I can take on this task, assign me please