Skip to content
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

Add the ability to block assigning ranks that allow players to become battle commanders during sieges. #950

Merged
merged 3 commits into from
Mar 9, 2025

Conversation

LlmDl
Copy link
Member

@LlmDl LlmDl commented Mar 8, 2025

Description:

Can be during any siege, or during only active battle sessions.


New Nodes/Commands/ConfigOptions:

New Config Nodes:

	BATTLE_COMMANDERS_BLOCK_ASSIGNMENT_DURING_BATTLE_SESSIONS(
			"battle_commanders.block_rank_assignment_during_active_battlesessions",
			"false",
			"",
			"# If this value is true, then the ranks containing the
siegewar.nation.siege.startconquestsiege permission node",
			"# cannot be assigned to players who are part of a siege with an
active battle session."),

	BATTLE_COMMANDERS_BLOCK_ASSIGNMENT_DURING_SIEGES(
			"battle_commanders.block_rank_assignment_during_sieges",
			"false",
			"",
			"# If this value is true, then the ranks containing the
siegewar.nation.siege.startconquestsiege permission node",
			"# cannot be assigned to players who are part of a siege."),

Relevant Issue ticket:

Closes #949.


  • 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.

battle commanders during sieges.

Can be during any siege, or during only active battle sessions.

Closes #949.

New Config Nodes:

	BATTLE_COMMANDERS_BLOCK_ASSIGNMENT_DURING_BATTLE_SESSIONS(
			"battle_commanders.block_rank_assignment_during_active_battlesessions",
			"false",
			"",
			"# If this value is true, then the ranks containing the
siegewar.nation.siege.startconquestsiege permission node",
			"# cannot be assigned to players who are part of a siege with an
active battle session."),

	BATTLE_COMMANDERS_BLOCK_ASSIGNMENT_DURING_SIEGES(
			"battle_commanders.block_rank_assignment_during_sieges",
			"false",
			"",
			"# If this value is true, then the ranks containing the
siegewar.nation.siege.startconquestsiege permission node",
			"# cannot be assigned to players who are part of a siege."),
@LlmDl LlmDl added this to the 2.16.0 milestone Mar 8, 2025
@LlmDl LlmDl merged commit 672ca7f into master Mar 9, 2025
@LlmDl LlmDl deleted the feat/block_commander_rank_adding_during_sieges branch March 9, 2025 14:25
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: cannot assign different siege commanders during the war
1 participant