Skip to content

feat: crash music volume config - #2076

Open
rapbattlegod32 wants to merge 3 commits into
mainfrom
feat/crash-music-config
Open

rapbattlegod32 wants to merge 3 commits into
mainfrom
feat/crash-music-config

Conversation

@rapbattlegod32

Copy link
Copy Markdown
Member

About the PR

add volume config for the new crash music

Why / Balance

too loud imo

Requirements

🆑

  • add: Volume config for the crash music!

@rapbattlegod32
rapbattlegod32 requested a review from a team as a code owner June 9, 2026 03:45
@github-actions github-actions Bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. A: Datagen Area: Datagen implementation & API. A: Tardis Components Area: Tardis components & manager. S: Needs Review Status: Requires additional reviews before being fully accepted. size/S Denotes a PR that changes 10-99 lines. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Jun 9, 2026
@GSMPBot
GSMPBot requested a review from Loqor June 9, 2026 03:45
Comment thread src/main/java/dev/amble/ait/core/tardis/handler/travel/CrashableTardisTravel.java Outdated
@github-actions github-actions Bot added S: Awaiting Changes Status: Changes are required before another review can happen. and removed S: Needs Review Status: Requires additional reviews before being fully accepted. labels Jun 9, 2026
@drtheodor
drtheodor self-requested a review June 11, 2026 08:45
@github-actions github-actions Bot added S: Needs Review Status: Requires additional reviews before being fully accepted. and removed S: Awaiting Changes Status: Changes are required before another review can happen. labels Jun 11, 2026
Comment on lines +188 to +189
ClientPlayNetworking.registerGlobalReceiver(TravelHandler.CRASH_MUSIC, (client, handler, buf, responseSender) ->
client.execute(ClientCrashMusicSoundHandler::play));

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

absolutely not
see how flight sound and alarms are played

@github-actions github-actions Bot added S: Awaiting Changes Status: Changes are required before another review can happen. and removed S: Needs Review Status: Requires additional reviews before being fully accepted. labels Jun 11, 2026
return tardis != null && !tardis.crash().isNormal();
}

public void tick(MinecraftClient client) {
@github-actions github-actions Bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted. label Jun 15, 2026
@github-actions

Copy link
Copy Markdown

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@rapbattlegod32
rapbattlegod32 requested a review from drtheodor June 28, 2026 01:22
@github-actions github-actions Bot added S: Needs Review Status: Requires additional reviews before being fully accepted. and removed S: Awaiting Changes Status: Changes are required before another review can happen. labels Jun 28, 2026

@drtheodor drtheodor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

does it work (tested)?

@drtheodor
drtheodor self-requested a review June 28, 2026 03:25

@drtheodor drtheodor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: Datagen Area: Datagen implementation & API. A: Tardis Components Area: Tardis components & manager. S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted. S: Needs Review Status: Requires additional reviews before being fully accepted. size/S Denotes a PR that changes 10-99 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants