Skip to content

Move away from adventure-platform for Paper #6220

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

Open
wants to merge 4 commits into
base: 2.x
Choose a base branch
from

Conversation

JRoy
Copy link
Member

@JRoy JRoy commented Jul 14, 2025

Makes us no longer dependent on adventure-platform for Paper servers as we will use their native adventure apis.

Few hacks needed to be done to not remap things in places but this is the best we can do

The PaperAdventureFacet has to be in PaperProvider to avoid the kyori remapping that happens on the entire EssentialsX module (we already did this for PaperChatListenerProvider)

Fixes #6215

@JRoy JRoy added this to the 2.21.2 milestone Jul 14, 2025
@sarahkittyy
Copy link

i pulled this pr's build artifact and i still cannot click on minimessage clickable links / see the hover msg

@JRoy
Copy link
Member Author

JRoy commented Jul 16, 2025

i pulled this pr's build artifact and i still cannot click on minimessage clickable links / see the hover msg

need an /ess dump all

@sarahkittyy
Copy link

@JRoy
Copy link
Member Author

JRoy commented Jul 16, 2025

essentialsx.net/dump.html?bytebin=j8AB75k9zg

thank you, can i also get the translation you are using (both the key and the actual translation line)

@sarahkittyy
Copy link

@JRoy how do i get that for you? i'm not familiar with this project i am just a user :)

@JRoy
Copy link
Member Author

JRoy commented Jul 16, 2025

@JRoy how do i get that for you? i'm not familiar with this project i am just a user :)

it would be any files in the plugins/Essentials/messages folder. This is where you would customize the messages to add hover events in the first place.

@sarahkittyy
Copy link

It's empty. I'm using https://github.com/HappyAreaBean/SimpleJoinMessage/ however the issue persists with /tellraw output as well

@JRoy
Copy link
Member Author

JRoy commented Jul 16, 2025

It's empty. I'm using HappyAreaBean/SimpleJoinMessage however the issue persists with /tellraw output as well

so you're saying that when you have essentials installed, sending a hover message with /tellraw doesn't even work?

@sarahkittyy
Copy link

yep!
image

/tellraw @a {"text":"testing 123","hoverEvent":{"action":"show_text","contents":"hover text"}}

@JRoy
Copy link
Member Author

JRoy commented Jul 20, 2025

yep! image

/tellraw @a {"text":"testing 123","hoverEvent":{"action":"show_text","contents":"hover text"}}

This has nothing to do with Essentials, Mojang updated the format for JSON messages, so you need to do:

/tellraw @a {"text":"testing 123","hover_event":{"action":"show_text","value":"hover text"}}

JRoy added 2 commits July 19, 2025 18:58
# Conflicts:
#	EssentialsChat/src/main/java/com/earth2me/essentials/chat/processing/AbstractChatHandler.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some MiniMessage tags are not working
2 participants