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 Outpost object. #7660

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add Outpost object. #7660

wants to merge 4 commits into from

Conversation

LlmDl
Copy link
Member

@LlmDl LlmDl commented Nov 4, 2024

Description:

Big update to outposts, creating a proper object from which we can track their size.

TODO:

  • handle /plot set outpost, re: do we care about sizing/filling out the outposts' townblocks.
  • handle importing legacy outposts.
  • probably more stuff.
  • reduce /plot perm hud to 15 lines.

New Nodes/Commands/ConfigOptions:

To Add:

  • town_level based outposts sizing
  • non-town_level based flat rate max sizing

Relevant Towny Issue ticket:

Closes #7547


  • 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 Towny License for perpetuity.

Remove old outpost db stuff from sql source.
Start on replacing old town methods using outpostSpawns with new stuff.
Add support for moving outpost spawns, setting outposts with /plot set
outpost.
Add a maybe-working legacy outpost converter.
Make the legacy converter twice as likely to succeed.
Lots of testing completed, legacy conversion working well.

/plot set outpost spawn & /plot set outpost taken care of.

TODO: Change Plot Perm Hud to reduce lines to 15.
Better logic and message on claim denial.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Suggestion: Outpost Claim Limit Size
1 participant