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

Suggestion: Create new District TownyObject. #7479

Closed
LlmDl opened this issue Jun 20, 2024 · 4 comments · Fixed by #7480
Closed

Suggestion: Create new District TownyObject. #7479

LlmDl opened this issue Jun 20, 2024 · 4 comments · Fixed by #7480
Assignees
Milestone

Comments

@LlmDl
Copy link
Member

LlmDl commented Jun 20, 2024

  • Extends our ObjectGroup, like a PlotGroup.

  • Similar command structure to PlotGroup, but with limited fields: UUID uuid, String name, Set<TownBlock> townblocks

  • Similar database to PlotGroup.

  • Allows a mayor/assistant to create a grouping of plots into what is named a District.

    • Districts have a name, and a Set of TownBlocks.
    • These District names will appear in Chunk Notifications, on the Dynmap.
  • Unlike PlotGroups, the townblocks within a District are allowed to have their individual plot permissions, owners and plot names.

The end goal is to make a blob of land which can be given a distinct name, without requiring the use of restrictive PlotGroups.

This could be used to name a blob of land surrounding an Outpost townblock, and so this concept will supercede the Suggestion: Set outpost names ticket.

@LlmDl
Copy link
Member Author

LlmDl commented Jun 20, 2024

These districts should probably try to reuse the adjacent claims code we have, so they are forced into being contiguous land masses.

@LlmDl LlmDl self-assigned this Jun 20, 2024
@MilesBHuff
Copy link

MilesBHuff commented Jun 20, 2024

I know you mentioned in Discord that you weren't sure how to handle these on the map. Maybe there should be a separate ticket for any mapping aspects, to allow this feature to go through faster, without being held-back?

@LlmDl
Copy link
Member Author

LlmDl commented Jun 21, 2024

Nothing will happen on the mapping side until I get around to making Dynmap-Towny aware of Districts.

@LlmDl LlmDl added this to the 0.100.4.0 milestone Jul 12, 2024
LlmDl added a commit that referenced this issue Jul 12, 2024
    - Districts are groupings of townblocks, owned by a town.
    - These are similar to PlotGroups but with less restrictions:
      - Districts' townblocks are not required to have the same plot
permissions, plot type, owner, or name.
    - Districts currently show in the chunk notifications, map hud, and
in the /towny map hover.
    - Districts will lead to future features being possible including
districts appearing on Mappers.
    - Townblocks in districts are required to be adjacent to each other.
    - Closes #7479.
  - New Commands: /plot district
    - Subcommands:
      - create|new [name]: creates a new district with the given name.
      - add {name}: adds a townblock to the given district, if you are
actively adding to a distric you do not need to continue supplying the
name.
      - rename [name]: changes the name of a district.
      - remove: removes the townblock you are stood in from a district.
      - delete: deletes the district in which you are stood.
  - New Permission Node: towny.command.plot.district.*
    - Child node of towny.command.plot.*
    - Child Nodes:
      - towny.command.plot.district.add: true
      - towny.command.plot.district.remove: true
      - towny.command.plot.district.delete: true
      - towny.command.plot.district.rename: true
@MilesBHuff
Copy link

You rock, Llmdl!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants