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

Remove old sculk spread option #6189

Merged
merged 1 commit into from
Sep 24, 2022
Merged

Conversation

Warriorrrr
Copy link
Member

Description:


New Nodes/Commands/ConfigOptions:


Relevant Towny Issue ticket:


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

@LlmDl LlmDl merged commit f640455 into master Sep 24, 2022
@LlmDl LlmDl deleted the feat/remove-legacy-skulk-option branch September 24, 2022 14:18
LlmDl added a commit that referenced this pull request Sep 24, 2022
PR #6190.
  - Remove old sculk spread option, courtesy of Warriorrrr with
PR #6189.
  - Town board command improvements, courtesy of Warriorrrr with
PR #6186.
  - Replace ChatTools.stripColour with Colors.strip, courtesy of
Warriorrrr with PR #6185.
  - API: Add Confirmation related events, courtesy of Warriorrrr with
PR #6182.
    - ConfirmationEvent
    - ConfirmationCancelEvent
    - ConfirmationConfirmEvent
    - ConfirmationSendEvent
      - Includes the ability "hide" the confirmation for plugin authors
to replace with their own selection/display.
  - API: Add a ConfirmationTransaction to Confirmations.
    - This enables cost-requirements to be baked-in to Confirmations.
    - Costs are passed in as a Supplier<Double> so that calculations can
be made after the player accepts.
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.

2 participants