Migrate from Spigot to Paper! - #228
Open
BlitzOffline wants to merge 18 commits into
Open
Conversation
|
Edit the plugin.yml to make sur the plugin is in the "Paper Plugins:" list when you type |
|
And why Java17 and not Java21? |
Member
Author
This is a good point. I think we are free to move to Java 21, even if we continue supporting 1.17.1+. I will make this change. |
…otes for later (regarding some API features that might break the plugin on older versions of paper), Make use of the paper-plugin.yml file
Contributor
|
Hey @BlitzOffline, would you like me to PR my changes (ignore the open/close commands + dupe protection stuff) to this branch?: main...GemstoneGG:DeluxeMenus:master This fixes loading and covers more JX -> J21 stuff |
R00tB33rMan
added a commit
to GemstoneGG/DeluxeMenus
that referenced
this pull request
Jul 26, 2026
This resupplies Folia support that was reverted in: 39ecf3a Notable changes (this should be tested by maintainers for confidence): - HelpChat#289 (Resolves) - HelpChat#285 (Resolves) - Implements DeluxeMenus's Paper compatibility layer slightly more extensively than: HelpChat#228 (minus MiniMessage compatibility layer due to potential convolusion) - Bumps a few dependencies (including PlaceholderAPI) - Hardens dupe mitigation - Introduces open + close command layering - Full Java 21 conversion If Folia issues are encountered and this is partially or completely merged, feel free to @ (ping) me, and I'm happy to investigate. I wasn't aware that reversion was on the table roughly a month ago!
R00tB33rMan
added a commit
to GemstoneGG/DeluxeMenus
that referenced
this pull request
Aug 11, 2026
This resupplies Folia support that was reverted in: 39ecf3a Notable changes (this should be tested by maintainers for confidence): - HelpChat#289 (Resolves) - HelpChat#285 (Resolves) - Implements DeluxeMenus's Paper compatibility layer slightly more extensively than: HelpChat#228 (minus MiniMessage compatibility layer due to potential convolusion) - Bumps a few dependencies (including PlaceholderAPI) - Hardens dupe mitigation - Introduces open + close command layering - Full Java 21 conversion If Folia issues are encountered and this is partially or completely merged, feel free to @ (ping) me, and I'm happy to investigate. I wasn't aware that reversion was on the table roughly a month ago!
…p reflection to Bukkit#getCommandMap for menu commands
BlitzOffline
marked this pull request as ready for review
August 17, 2026 17:28
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.
I am creating this PR so people can keep track of our effort on migrating to Paper-Only.
If you have some ideas or suggestions, feel free to leave a comment under this PR.
Things done so far:
suppress_default_italics)'Here is a list of tickets that are related to this PR and could be solved using this PR: #10, #15 and #172
Testing:
I have manually tested colors and formatting: menu titles, item lore and display name, lore append mode with ItemsAdder
I have also manually tested heads: basehead, texture head, head has item requirements
I have tested the dm command for both execution and completion
I have tested command registration (it works just like before, without ability to register commands only on server startup)
I have tested most actions, including ones related to messaging, as well as player and commandevent actions.
I have published a thread in Discord asking people for help with testing. Even if the PR gets the required approvals, we will wait some time before merging, to make sure people that were able to help with testing have made any necessary reports.