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

Respect per player locale for command descriptions #5972

Merged
merged 7 commits into from
Nov 24, 2024
Merged

Conversation

iiAhmedYT
Copy link
Contributor

Information

This PR fixes #5841

Details

Proposed fix:
Simply use the already made per language description per command

Environments tested:

OS: Windows

Java version:

  • Most recent Paper version (1.XX.Y, git-Paper-BUILD)
  • CraftBukkit/Spigot/Paper 1.12.2
  • CraftBukkit 1.8.8

Demonstration:

Before:
Before

After:
After

@JRoy JRoy requested a review from Copilot November 24, 2024 01:46

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no suggestions.

Copy link
Member

@JRoy JRoy left a comment

Choose a reason for hiding this comment

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

This will not work if components are

iiAhmedYT and others added 2 commits November 24, 2024 12:41
Co-authored-by: Josh Roy <10731363+JRoy@users.noreply.github.com>
@iiAhmedYT
Copy link
Contributor Author

Will continue this once im home (im like 5 hours away)

@iiAhmedYT iiAhmedYT requested a review from JRoy November 24, 2024 18:56
Copy link
Member

@JRoy JRoy left a comment

Choose a reason for hiding this comment

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

few things and then should be good

@iiAhmedYT iiAhmedYT requested a review from JRoy November 24, 2024 22:16
@JRoy JRoy changed the title Respect user locale in Help message descriptions Respect per player locale for command descriptions Nov 24, 2024
@JRoy JRoy enabled auto-merge (squash) November 24, 2024 22:23
@JRoy
Copy link
Member

JRoy commented Nov 24, 2024

Thanks for the contribution :)

@JRoy JRoy merged commit 1778bf5 into EssentialsX:2.x Nov 24, 2024
1 check passed
Euphillya added a commit to Euphillya/Essentials-Folia that referenced this pull request Dec 10, 2024
EssentialsX@1778bf5 Respect per player locale for command descriptions (EssentialsX#5972)
EssentialsX@2418a6f Improvements to Random Teleport (EssentialsX#4271)
EssentialsX@38e42f9 Add argument to /skull to give skull to other player (EssentialsX#5822)
EssentialsX@1af1565 Discord: Add start time placeholder for start message (EssentialsX#5686)
EssentialsX@bea43e8 Fix mini message parsing in NumberUtil#displayCurrency (EssentialsX#5921)
EssentialsX@3203e97 Parse /pay modifiers more strictly (EssentialsX#5638)
EssentialsX@f02eeec Add method to get all linked players in DiscordLinkService (EssentialsX#5896)
EssentialsX@9590d4c Add translation key for /alts output (EssentialsX#5595)
EssentialsX@3f0a412 Add locale based comma support for /pay (EssentialsX#5962)
EssentialsX@57c9edc Add optional requirements for balance top listing (EssentialsX#5394)
EssentialsX@c7cc1b4 Fix respawn-at-anchor setting. (EssentialsX#5825)
EssentialsX@2a41ea0 Add config options for specific chat type formats
EssentialsX@b560bbd Fix actions permissions on PRs for junit reports
EssentialsX@d31ff55 Remove extraneous code in command preprocess event handling
EssentialsX@bc8aa76 Revert removal of InvalidWorldException (EssentialsX#5984)
EssentialsX@d58db0d Update to 1.21.4
EssentialsX@c37064f Add 1.21.4 Mobs
EssentialsX@baa8258 Add 1.21.4 Tree Types
EssentialsX@424816e Add 1.21.4 Items
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.

Command Description string do not respect locale when not exist in the message config
2 participants