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

Feature/cloud commands #38

Merged
merged 8 commits into from
Jul 28, 2024
Merged

Feature/cloud commands #38

merged 8 commits into from
Jul 28, 2024

Conversation

JvstvsHD
Copy link
Owner

@JvstvsHD JvstvsHD commented Jul 21, 2024

This will implement the contents of #11.
Not yet ready for production/testing, as some features are still missing:

  • General structure for registering commands via the cloud framework platform-independently
  • config option to disallow registration of top-level-commands (/ban... instead of /necrify ban...)
  • Ban and mute command
  • Tempban and tempmute command
  • Unmute/unban command
  • Punishment info command
  • Playerinfo command

Add a base structure for all commands
# Conflicts:
#	necrify-velocity/build.gradle.kts
@JvstvsHD JvstvsHD marked this pull request as draft July 21, 2024 16:56
@JvstvsHD JvstvsHD self-assigned this Jul 21, 2024
- Added dedicated console user
- Added more methods to send messages easier
- Added documentation in CommandSender.java
- Fixed some smaller bugs
API: Change return type of punishing methods to CompletableFuture<...>, thus exceptions are no longer swallowed and be easily checked by a handler.
MiniMessage is now being used to serialize reasons when sending over messaging channels
Fixed some smaller issues
Also added some nice preview completions that will show you the expiration date and the formatted MiniMessage when hovering over the completion
@JvstvsHD JvstvsHD linked an issue Jul 28, 2024 that may be closed by this pull request
Split original /punishment command into:
- /necrify punishment <UUID> info|cancel|remove|change
- /necrify user <user> info|delete
/necrify punishment and /necrify user delete are currently not usable since further features are missing
@JvstvsHD JvstvsHD marked this pull request as ready for review July 28, 2024 17:25
@JvstvsHD JvstvsHD merged commit 194fd93 into master Jul 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Migrate commands into single command
1 participant