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

ESSX Discord doesn't respect announceAdvancements gamerule when sending advancement message #4865

Open
JellyBlade opened this issue Mar 22, 2022 · 1 comment
Labels
module: discord Issues or PRs for the EssentialsDiscord module type: enhancement Features and feature requests.
Milestone

Comments

@JellyBlade
Copy link

Type of bug

Other unexpected behaviour

/ess dump all output

https://essentialsx.net/dump.html?id=638bef65de1c46f38e109c26a6d2a24a

Error log (if applicable)

No response

Bug description

Disabling the announceAdvancement gamerule still allows the advancement message through to Discord. Current server setup has two worlds, creative and survival.

I created a plugin to revoke advancements made while in creative mode (simplest solution without modifying Multiverse) and disabled announceAdvancements in the creative world, but the message gets displayed in the primary discord channel anyway.

Ideally, advancement messages would still be enabled for the survival world, so disabling advancement messages on the bridge entirely is something I'd like to avoid.

Checking for whether or not the player's current world has the announceAdvancements gamerule enabled before sending the message would prevent this behaviour from occurring.

Steps to reproduce

  1. Disable announceAdvancements gamerule.
  2. Complete an advancement.
  3. Message still gets displayed in the channel, even though the message for in-game chat is disabled.

Expected behaviour

Having the announceAdvancements gamerule disabled should prevent the message from appearing in discord.

Actual behaviour

The message is displayed regardless of the player's current world's announceAdvancements gamerule value.

@JellyBlade JellyBlade added the bug: unconfirmed Potential bugs that need replicating to verify. label Mar 22, 2022
@pop4959 pop4959 added type: enhancement Features and feature requests. and removed bug: unconfirmed Potential bugs that need replicating to verify. labels May 22, 2022
@BornToDoStuff
Copy link

This bug also applies when using datapacks to control advancements such as Blaze and Caves advancements. The advancement announcement event is successfully canceled and does not appear in chat, to the player, or to the console, but is announced through the Discord connection regardless. I imagine the root reason is extremely similar or identical to the gamerule bug.

@mdcfe mdcfe added this to the 2.21.0 milestone Jul 15, 2023
@mdcfe mdcfe modified the milestones: 2.21.0, 2.21.1 Sep 15, 2024
@mdcfe mdcfe added the module: discord Issues or PRs for the EssentialsDiscord module label Sep 15, 2024
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: enhancement Features and feature requests.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants