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.

Sign up for GitHub

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

Jump to bottom

Suggestion: Add the ability for towns to toggle whether plots go up for sale on resident leave/purge #7295

Closed
CorruptedGreed opened this issue Mar 1, 2024 · 3 comments · Fixed by #7313
Closed

Suggestion: Add the ability for towns to toggle whether plots go up for sale on resident leave/purge #7295

CorruptedGreed opened this issue Mar 1, 2024 · 3 comments · Fixed by #7313
Assignees
@Warriorrrr
Labels
Completed enhancement Label-TownBlock
Milestone
0.100.3.0

Comments

@CorruptedGreed
Copy link

Please explain your feature request to the best of your abilities:

Add a command (such as /t toggle plotresale) for towns to determine whether plots put themselves back for sale upon a resident leaving/purging.

Currently towns circumvent unwanted purchases of plots put back for sale by setting high plotprice (i.e /t set plotprice 10000). Having the ability to toggle plotresale would just result in plots being reclaimed by the town and nfs.
@LlmDl
Copy link
Member

LlmDl commented Mar 1, 2024

I do like the idea of making this a thing for mayors to decide but I thought I'd also make sure you're aware of a global setting that does this already:

"town.default_taxes.does_non-payment_place_plot_for_sale",
"false",
"",
"# Does a player's plot get put up for sale if they are unable to pay the plot tax?",
"# When false the plot becomes town land and must be set up for-sale by town mayor or staff."),

@CorruptedGreed
Copy link
Author

Would enabling this work with taxes disabled, and does it only apply to plots put up for sale after failing to pay plot tax (or general removal)?

@LlmDl
Copy link
Member

LlmDl commented Mar 1, 2024

Yeah this setting is only used when plots are lost due to non-payment of taxes. I wasn't thinking about when players leave/are purged.

@Warriorrrr Warriorrrr mentioned this issue Mar 16, 2024
Merged
1 task
@LlmDl LlmDl added this to the 0.100.2.0 milestone Mar 19, 2024
@LlmDl LlmDl modified the milestones: 0.100.2.0, 0.100.3.0 Mar 29, 2024
@LlmDl LlmDl added the Completed label May 6, 2024
LlmDl added a commit that referenced this issue May 6, 2024
@LlmDl
- Fix workflows running twice on pull requests, courtesy of Warrior
69ab200 
with PR #7402.
  - Fix Towny not loading on older minecraft servers missing
FileNameUtils modern methods, courtesy of Warrior with PR #7403.
    - Closes #7399.
  - Allow setting a default plot price of -1, courtesy of Warrior with
PR #7313.
    - A town can now prevent their plots going up for sale by setting a
plot price of -1, using /t set plotprice -1.
    - Closes #7295.
  - Add option to use world key instead of world name, courtesy of
gecko10000 with PR #7388. (First-Time Contributor!)
  - New Config Option:
plugin.interfacing.web_map.world_name_uses_world_key
    - Default: false
    - If enabled, the world name placeholder will be replaced with the
world key instead of the Bukkit name.
    - This should be enabled if you use SquareMap.
  - Bump adventure platform bukkit to 4.3.3-SNAPSHOT, courtesy of
Warrior with PR #7395.
    - Adds support for 1.20.5/6 in Towny.
  - API: add NATION_LIST_BY, TOWN_LIST_BY TownyCommandAddons.
    - Closes #7397.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Warriorrrr Warriorrrr

Labels
Completed enhancement Label-TownBlock
Projects
None yet
Milestone
0.100.3.0
Development

Successfully merging a pull request may close this issue.

Allow setting a default plot price of -1 TownyAdvanced/Towny
3 participants
@LlmDl @CorruptedGreed @Warriorrrr