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

We're going on an Adventure! #4717

Merged
merged 129 commits into from
Feb 3, 2024
Merged

We're going on an Adventure! #4717

merged 129 commits into from
Feb 3, 2024

Conversation

JRoy
Copy link
Member

@JRoy JRoy commented Dec 28, 2021

im.going.on.an.adventure.mp4

Closes #2029
Closes #2391

Todo

  • Per Player Locale
    • Add support for them
    • Finish porting everything over to per-player-locale system.
  • MiniMessage Support (Components - Hover/Click Events)
    • No messages will be converted to MiniMessage in this PR, a separate PR will add this.
    • Currently, there are a few messages which are a giant pain in the ass to convert to support components. They will be converted on a need-to basis.

@JRoy JRoy added type: enhancement Features and feature requests. module: main Issues or PRs for the main Essentials module module: discord Issues or PRs for the EssentialsDiscord module module: chat Issues or PRs for the EssentialsChat module module: spawn Issues or PRs for the EssentialsSpawn module module: xmpp Issues or PRs for the EssentialsXMPP module module: geo Issues or PRs for the EssentialsGeo module module: protect Issues or PRs for the EssentialsProtect module labels Dec 28, 2021
@JRoy JRoy added this to the 2.20.0 milestone Dec 28, 2021
Turns out that adventure-platform registers events which are required to
create an audience for them. Move the creation of the BukkitAudiences
object after we unregister all of our events.
…is i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this i hate this
@JRoy JRoy force-pushed the refactor/we-are-going-on-an-adventure branch from 8c538dc to 2f38558 Compare December 28, 2021 23:19
…y head get out of my head get out of my head get out of my head get out of my head
… i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die i want to die
This reverts commit 90103ba.
@TheKrafter
Copy link

TheKrafter commented Nov 24, 2023

Just your friendly neighborhood server admin, I was wondering if we could get a progress report of some kind. I really appreciate yall's work, and also had a few questions regarding these changes that the answers to are probably well-known to EssentialsX developers, but I have not been able find answers to elsewhere:

I don't want to be a nag, but I really do appreciate your work and would like to know how this all is going so far, thank you all very much!

@codebycam
Copy link

LGTM

Still LGTM

mdcfe
mdcfe previously approved these changes Feb 3, 2024
@SrBedrock
Copy link
Contributor

SrBedrock commented Feb 3, 2024

When a player joins the server, an error is sent: https://pastes.dev/YbBHLlf2fw
When trying to receive messages from Essentials an error is sent: https://pastes.dev/lB3llBOiDM

image

@mbax
Copy link
Member

mbax commented Feb 3, 2024

When a player joins the server, an error is sent: https://pastes.dev/YbBHLlf2fw When trying to receive messages from Essentials an error is sent: https://pastes.dev/lB3llBOiDM

image

Any chance you were live swapping the EssX jar in the plugins folder instead of using plugins/update ?

@mdcfe
Copy link
Member

mdcfe commented Feb 3, 2024

@SrBedrock The PR has been updated to shade the missing options library - would you be able to update to the latest dev build?

@mdcfe mdcfe merged commit 388d571 into 2.x Feb 3, 2024
1 check passed
@mdcfe mdcfe deleted the refactor/we-are-going-on-an-adventure branch February 3, 2024 20:38
sarhatabaot added a commit to Lapzupi/Essentials that referenced this pull request Apr 6, 2024
HarvelsX pushed a commit to HarvelsX/Essentials that referenced this pull request Jun 2, 2024
https://user-images.githubusercontent.com/10731363/147530817-f9adc58e-18a5-49ed-84c6-106e51d6948f.mp4
Add support for chat components throughout EssentialsX using the Adventure library.
Translations have been converted to the MiniMessage format, and custom message files will be migrated on startup.
This also introduces new options to allow players to see messages in their own language and for server owners to change the main message colours without editing message files.

Closes EssentialsX#2029
Closes EssentialsX#2391

---------

Co-authored-by: MD <[email protected]>
Co-authored-by: pop4959 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: chat Issues or PRs for the EssentialsChat module module: discord Issues or PRs for the EssentialsDiscord module module: geo Issues or PRs for the EssentialsGeo module module: main Issues or PRs for the main Essentials module module: protect Issues or PRs for the EssentialsProtect module module: spawn Issues or PRs for the EssentialsSpawn module module: xmpp Issues or PRs for the EssentialsXMPP module type: enhancement Features and feature requests.
Projects
None yet