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 SiegeCamp object, created when a Siege is about to start. #404

Merged
merged 9 commits into from
Nov 18, 2021

Conversation

LlmDl
Copy link
Member

@LlmDl LlmDl commented Nov 17, 2021

Description:

Object collects all the information required for the
PreSiegeWarStartEvent and starting the actual Siege.

Upon creation, a SiegeCamp begins a 10 minute period where on each
minute if the attackers are present within the TimedPointZone, the
attackers gain 7 points. After the 10 minute period, if the Attackers
have 50 points the Siege will begin.

TODO:

  • Language strings.

  • Configuration section for points, times.

  • Make SiegeCamps optional, falling back to old system when disabled.

  • Comments.

  • Basic testing.

  • Protect the banner for the duration of the SiegeCamp session.

  • Check that the war chest can still be paid.


New Nodes/Commands/ConfigOptions:


Relevant Issue ticket:

Closes #402.


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

Object collects all the information required for the
PreSiegeWarStartEvent and starting the actual Siege.

Upon creation, a SiegeCamp begins a 10 minute period where on each
minute if the attackers are present within the TimedPointZone, the
attackers gain 7 points. After the 10 minute period, if the Attackers
have 50 points the Siege will begin.

Closes #402.

TODO:
- [ ] Language strings.
- [ ] Configuration section for points, times.
- [ ] Basic testing.
broadcasts informing server of SiegeCamp session.
@LlmDl LlmDl added this to the 0.6.5 milestone Nov 18, 2021
@LlmDl LlmDl marked this pull request as ready for review November 18, 2021 00:23
@LlmDl LlmDl self-assigned this Nov 18, 2021
Copy link
Member

@Warriorrrr Warriorrrr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, could potentially be expanded upon with some (pre) events.

@LlmDl LlmDl merged commit d86473c into master Nov 18, 2021
@LlmDl LlmDl deleted the feature/siege_camps branch November 18, 2021 20:20
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.

Preparing for a siege
3 participants