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

API: Add TOWNYADMIN_TOWN and TOWNYADMIN_NATION CommandTypes #6179

Merged
merged 3 commits into from
Sep 20, 2022

Conversation

Lemar98
Copy link
Contributor

@Lemar98 Lemar98 commented Sep 20, 2022

Added two new types for creating subcommands: TOWNYADMIN_TOWN and TOWNYADMIN_NATION which are triggered by /townyadmin town <town_name> and /townyadmin nation <nation_name> respectively


The purpose of this PR is to allow other developers to add their own subcommands for /townyadmin <town/nation> <government_name>. For example, /townyadmin town test mysubcommand.
I believe this will make the API more flexible


I tested this code on my local server paper 1.16.5


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.

Added two new types for creating subcommands: TOWNYADMIN_TOWN and TOWNYADMIN_NATION which are triggered by /townyadmin town <town_name> and /townyadmin nation <nation_name> respectively
Copy link
Member

@LlmDl LlmDl left a comment

Choose a reason for hiding this comment

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

All of the unneeded diffs need to be removed from this PR.

@LlmDl LlmDl changed the title Add command types for api API: Add TOWNYADMIN_TOWN and TOWNYADMIN_NATION CommandTypes Sep 20, 2022
remove free spaces
@Lemar98
Copy link
Contributor Author

Lemar98 commented Sep 20, 2022

Seems to be fixed

@LlmDl
Copy link
Member

LlmDl commented Sep 20, 2022

There's more, I'll fix it up for you.

@Lemar98
Copy link
Contributor Author

Lemar98 commented Sep 20, 2022

Thanks!

@LlmDl LlmDl merged commit 4f886f1 into TownyAdvanced:master Sep 20, 2022
LlmDl added a commit that referenced this pull request Sep 20, 2022
courtesy of Lemar98 with PR #6179. (First-Time Contributor!)
@LlmDl LlmDl added this to the 0.98.4.0 milestone Sep 20, 2022
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