Skip to content

Conversation

@JRoy
Copy link
Member

@JRoy JRoy commented Jul 27, 2025

Fixes #5878. Caused by a regression from 384f63b, we were checking for for the legacy options when we should really be checking if mc-to-discord-name-format is anything other than the bot name.

@JRoy JRoy added this to the 2.21.2 milestone Jul 27, 2025
@JRoy JRoy added type: bugfix PRs that fix bugs in EssentialsX. module: discord Issues or PRs for the EssentialsDiscord module labels Jul 27, 2025
@JRoy JRoy enabled auto-merge July 27, 2025 23:25
@JRoy JRoy added this pull request to the merge queue Jul 28, 2025
Merged via the queue into EssentialsX:2.x with commit f4cd1c9 Jul 28, 2025
1 check passed
@JRoy JRoy deleted the fix/5878 branch July 28, 2025 05:40
Euphillya added a commit to Euphillya/Essentials-Folia that referenced this pull request Jul 29, 2025
EssentialsX@9ef4fc3 Update to Minecraft 1.21.8 (EssentialsX#6224)
EssentialsX@0417624 Fix local chat spy formatting (EssentialsX#6226)
EssentialsX@ff7c952 Fix per-player locale for messages with arguments (EssentialsX#6227)
EssentialsX@d002c34 Add version data extraction (EssentialsX#6235)
EssentialsX@f4cd1c9 Discord: Fix custom bot name requiring show-avatar (EssentialsX#6239)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: discord Issues or PRs for the EssentialsDiscord module type: bugfix PRs that fix bugs in EssentialsX.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

avatar: true is required for mc-to-discord-name-format to take any effect

2 participants