Bannerlord-Coop-Team / BannerlordCoop

MIT License
265 stars 51 forks source link

MapEvent Sync #670

Open EgardA opened 1 year ago

EgardA commented 1 year ago

For New Contributors

How to contribute

Description

MapEvent.FinalizeEvent()

Sync needed for Settlement Ownership Change

Intended Design

Follow design of this to get a sense of how to do start out and then follow the messages to keep going :)

https://github.com/Bannerlord-Coop-Team/BannerlordCoop/blob/587e19101cbb14f398aef04f6599b117acc0e48f/source/GameInterface/Services/Settlements/Patches/ChangeOwnerOfSettlementPatch.cs

Location

Follow above as well.

Related Issues

Requirements

Create an integration test for your messages

Additional information

Ask in contrib-chat if you require any assistance, we are happy to help you out.

Definition of Done

EgardA commented 10 months ago

This is more complicated than meets the eye after some testing