Skip to content

πŸ›βœ¨ Bugfixes & smaller enhancements#93

Merged
Zoriot merged 9 commits into
mainfrom
bugfix
Jun 26, 2026
Merged

πŸ›βœ¨ Bugfixes & smaller enhancements#93
Zoriot merged 9 commits into
mainfrom
bugfix

Conversation

@Zoriot

@Zoriot Zoriot commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

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.

Zoriot and others added 8 commits June 25, 2026 12:18
…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.
@Zoriot Zoriot requested a review from a team June 26, 2026 16:42
@Zoriot Zoriot self-assigned this Jun 26, 2026
@Zoriot Zoriot added 🚨 high priority This currently has high priority. 🐘 large size This is a large issue. labels Jun 26, 2026
@Zoriot Zoriot added this to the Release v0.3.0 milestone Jun 26, 2026
@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown

Qodana for JVM

It 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
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@Jasupa Jasupa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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 Jasupa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't have anything to add. Looks good to me.

@Zoriot Zoriot merged commit 1aea27a into main Jun 26, 2026
4 checks passed
@Zoriot Zoriot deleted the bugfix branch June 26, 2026 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 high priority This currently has high priority. 🐘 large size This is a large issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants