Bannerlord-Coop-Team / BannerlordCoop

MIT License
260 stars 51 forks source link

ItemObject Field Sync #995

Open EgardA opened 2 hours ago

EgardA commented 2 hours 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

Synced

Method/Field Name Sync Type
TODO TODO

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

EgardA commented 2 hours ago

I am not sure if we want to sync all of these, except name, and can instead base all properties/fields on the string id (except name, which seems to be customizeable)