Skip to content

feat: /feedback command (bundle Discord Presence 0.24.0) - #422

Open
bh679 wants to merge 1 commit into
mainfrom
claude/vibrant-wright-b92bff
Open

feat: /feedback command (bundle Discord Presence 0.24.0)#422
bh679 wants to merge 1 commit into
mainfrom
claude/vibrant-wright-b92bff

Conversation

@bh679

@bh679 bh679 commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Summary

Brings the new /feedback command to Dungeon Train by bundling Discord Presence 0.24.0 (jarJar). Any player can type /feedback — in any game mode — to open the feedback survey on demand, and re-submit. Previously the survey only appeared on the death screen (unreachable in creative/spectator).

discordpresence_version 0.22.0 → 0.24.0. Also pulls DP 0.23.0 (advancement message-suffix seam + account-link credential plumbing).

Mod impact

  • No DT code change — dependency version bump only. DP is jarJar'd (not a modpack optional_mods include), so no check-relations.py / modpack changes.
  • The new command + networking packet ship inside bundled DP (DP-side, already released as v0.24.0).

Test plan

  • ./gradlew build — clean; DT jar bundles discordpresence-0.24.0.jar; DT unit tests pass
  • DT dev client integration: loads clean, survey active (webhookSet/consent), /feedback registered in a creative world (Gate 2 approved)
  • DP v0.24.0 published (GitHub + Modrinth + CurseForge); DT resolves it via the GitHub-release Ivy repo
  • (post-merge) optional DT release to ship /feedback to players

🤖 Generated with Claude Code

Bumps discordpresence_version 0.22.0 -> 0.24.0. DP 0.24.0 adds an in-game /feedback command: any player, any game mode, can open the feedback survey on demand (re-submittable), not just on the death screen. Also pulls DP 0.23.0 (advancement message-suffix seam + account-link credential plumbing).

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
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.

1 participant