Skip to content

Fixed bug where non-participants could be in besieged town without full war-sickness#985

Merged
Goosius1 merged 2 commits intoTownyAdvanced:masterfrom
Goosius1:fix_minor_war_sickness
Oct 2, 2025
Merged

Fixed bug where non-participants could be in besieged town without full war-sickness#985
Goosius1 merged 2 commits intoTownyAdvanced:masterfrom
Goosius1:fix_minor_war_sickness

Conversation

@Goosius1
Copy link
Copy Markdown
Collaborator

@Goosius1 Goosius1 commented Oct 2, 2025

Description:

  • With current code, apparently non-participants in the besieged town SOMETIMES get full war sickness, and sometimes they do not
  • This PR fixes the issue, so that non-participants in the besieged town ALWAYS get full war sickness

New Nodes/Commands/ConfigOptions:

N/A


Relevant Issue ticket:

Closes #893


  • [ N/A Hopefully too trivial ] I have tested this pull request for defects on a server.

By making this pull request, I represent that I have the right to waive copyright and related rights to my contribution, and agree that all copyright and related rights in my contributions are waived, and I acknowledge that the TownyAdvanced organization has the copyright to use and modify my contribution under the SiegeWar License for perpetuity.

@Goosius1 Goosius1 self-assigned this Oct 2, 2025
@Goosius1 Goosius1 marked this pull request as ready for review October 2, 2025 10:18
@Goosius1 Goosius1 added this to the 3.0.0 milestone Oct 2, 2025
@Goosius1 Goosius1 merged commit b5009df into TownyAdvanced:master Oct 2, 2025
@Goosius1 Goosius1 deleted the fix_minor_war_sickness branch October 2, 2025 13:45
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.

Suggestion: Allow modifying the War Sickness feature

2 participants