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

Advancements are named after their file name not their title #1601

Open
2 tasks done
lerokko opened this issue Oct 20, 2023 · 6 comments · May be fixed by #1629
Open
2 tasks done

Advancements are named after their file name not their title #1601

lerokko opened this issue Oct 20, 2023 · 6 comments · May be fixed by #1629

Comments

@lerokko
Copy link

lerokko commented Oct 20, 2023

Expected behavior

DiscrodSRV uses the "title" in custom advancements for their message. To have parity with how the vanilla chat would display it.

Actual behavior

I have a custom advancement called root.json with the title "Welcome to the Server". DiscordSRV used to use this title for the advancement message but in the latest snapshot under 1.20.2 (paper-241) its seem to use the file name and make the first letter upper case "Root" (note that the word "Root" is nowhere in the json its just the file name.)

Steps to reproduce

Get the latest paper for 1.20.2 241 and discordSRV snapshot fdcb09f
Add the example datapack to the server. The advancement should be automatically granted on join and show up in chat and the advancement tab.
EDIT 2024: Uploaded the file to github so I can finally stop self-hosting it, lol. I did not know you could attach files to comments server26advancements_discordsrvtest.zip

Server software and version

git-Paper-241 (MC: 1.20.2)

Checks

  • I am not using an outdated version of DiscordSRV.
  • I asked in DiscordSRV's Discord server to see whether this issue is in fact a bug that needs to be fixed.

Anything else

No response

@lerokko lerokko added the bug label Oct 20, 2023
@Vankka Vankka changed the title Custom advancements are named after their file name not their title Advancements are named after their file name not their title Oct 30, 2023
@comicsansgreenki
Copy link

comicsansgreenki commented Oct 30, 2023

Issue seems present in two servers with stable build (1.26.2) - except they're vanilla advancements.

[13:35:58 INFO]: comicsansgreenki issued server command: /advancement grant @s only minecraft:nether/return_to_sender
[13:35:59 INFO]: comicsansgreenki has completed the challenge [Return to Sender]
[13:35:59 INFO]: [comicsansgreenki: Granted the advancement [Return to Sender] to comicsansgreenki]
[13:35:59 INFO]: [DiscordSRV] [MINECRAFT_TO_DISCORD DEBUG] Failed to get title of advancement nether/return_to_sender: Failed to find AdvancementDisplay getter for advancement handle
[13:37:24 INFO]: comicsansgreenki issued server command: /advancement grant @s only minecraft:story/deflect_arrow
[13:37:24 INFO]: comicsansgreenki has made the advancement [Not Today, Thank You]
[13:37:24 INFO]: [comicsansgreenki: Granted the advancement [Not Today, Thank You] to comicsansgreenki]
[13:37:24 INFO]: [DiscordSRV] [MINECRAFT_TO_DISCORD DEBUG] Failed to get title of advancement story/deflect_arrow: Failed to find AdvancementDisplay getter for advancement handle

image

@comicsansgreenki
Copy link

Also present in 1.27.0-SNAPSHOT-b3a177f.

[13:57:04 INFO]: comicsansgreenki issued server command: /advancement grant @s only minecraft:story/deflect_arrow
[13:57:04 INFO]: comicsansgreenki has made the advancement [Not Today, Thank You]
[13:57:04 INFO]: [comicsansgreenki: Granted the advancement [Not Today, Thank You] to comicsansgreenki]
[13:57:04 INFO]: [DiscordSRV] [MINECRAFT_TO_DISCORD DEBUG] Failed to get title of advancement story/deflect_arrow: Failed to find AdvancementDisplay getter for advancement handle

image

@TheUnknownCod3r
Copy link

I asked about this in the discord almost 2 months ago, and got no answer regarding, so good to know which plugin was at issue finally. We had no errors in the console or logs.

@Scarsz Scarsz closed this as completed in 268b9a7 Jan 5, 2024
@comicsansgreenki
Copy link

comicsansgreenki commented Jan 5, 2024

yayyy! tysm @Scarsz @granny <3

@AAlchemy
Copy link

Sorry to necro a close issued, but this problem is back with build 1.27.0 on paper 1.20.4-389
Player has made the advancement Form Obsidian!

@granny
Copy link
Member

granny commented Jan 18, 2024

@AAlchemy fixed again in snapshot builds, https://snapshot.discordsrv.com/

@granny granny reopened this Jan 18, 2024
@granny granny linked a pull request Jan 18, 2024 that will close this issue
@granny granny added the fixed label Apr 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants