Conversation
β¦ncChatEvent For simplicity we still use just a string internally, not components. Serialized via PlainTextComponentSerializer.
β¦module handling - Added `Utils.sendNoPermissionMessage` for consistent permission error messaging. - Updated command logic to call the utility method for permission checks. - Improved cache handling by enabling disabled modules when necessary. - Adjusted debug mode toggle and messaging with on/off support. - Refined handling of missing modules during cache display. - Reload Network Cache if api key was added
- Introduced `visitor`, `builder`, and `admin` permission packs with appropriate child permissions. - Adjusted default permissions for `/blockpalette` command to `op`. It default's to op because we want the server to be fully in charge of managing permissions and to avoid confusion. Other plugins like FAWE also handles it that way.
This should fix the repeating timeout's when calling the Network Api. Long term we should investigate why it's taking so long to respond.
- Integrated PaperUpdater for cleaner and more efficient update management. - Added new `auto-update` configuration option (`enabled` by default). - Removed legacy updater logic and API calls to streamline code. - Updated `checkForUpdates` command for better user notifications and permissions handling. - Bumped `config-version` to 1.5. - Changed from Spigot to GitHub Releases - Add a /update command for manual updating - Simplified the BTT cmd a bit
- Added handling for individual warp deletions before removing a warp group. - Improved error recovery and logging for API failures. - Restructured menu navigation logic for better usability and slot management. - Simplified slot assignment logic in paginated menus and updated method naming for clarity.
β¦deletion + fix pagination - Introduced a confirmation menu before deleting Warp Groups. - Added options to either move warps to a fallback group or delete them entirely. - Updated `WarpGroupEditMenu` to invoke the confirmation menu when deleting groups with warps. - Refactored `deleteWarpGroup` in `BuildTeam` to support conditional deletion of child warps. - Improved menu navigation with enhanced backing functionality across menus. - Applied minor code cleanups and optimizations in warp group menus.
Qodana for JVMIt seems all right π No new problems were found according to the checks applied π‘ Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at qodana-support@jetbrains.com
|
Jasupa
reviewed
Jun 26, 2026
Jasupa
left a comment
Contributor
There was a problem hiding this comment.
It's almost completely right. Good job! Only a few small things.
- Improved `shouldUpdate` with `getVersionDifference` method supporting SemVer comparison. - Added optional source parameter for better logging. - Moved to `VersionComparator` for cleaner and more robust version checks. - Annotated several parameters and methods with `@NonNull` for improved null safety and clarity. - Switched `BuildTeamToolsCommand` to a cleaner switch-case structure for better command handling.
Jasupa
approved these changes
Jun 26, 2026
Jasupa
left a comment
Contributor
There was a problem hiding this comment.
I don't have anything to add. Looks good to me.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Look at the commits for detailed information, we'll switch fully to GitHub Release now.
In the future i'll split that up into multiple pr's. I decided to not do the other unnecessary maintenance burden to change all calls from legacy strings to components, for some part we also need a Alps Lib Change.
It's possible that smaller dependency updates + the version bump will still follow.
Because of the current nwapi problems where the plugin often times out i increased the timeout so it should hopefully work again.
The plugin now tries to enable modules depending on the nwapi when the reload config command is executed or the cache update cmd.