Skip to content

Conversation

@QThund
Copy link
Contributor

@QThund QThund commented Dec 16, 2025

Pull Request Description

What does this PR change?

Audio clip is now null when outcome animation does not have sound.

Test Instructions

Prerrequisites: Needs an emote that plays a sound in the start animation, no sound in one outcome and other sound in other outcome. Ask @QThund for it.

Outcome without sound

  1. Play start animation. Hear the sound repeating.
  2. React using other avatar, choosing the outcome without sound. Both play outcome animation. No sound plays.

Outcome with sound

  1. Play start animation. Hear the sound repeating.
  2. React using other avatar, choosing the outcome with sound. Both play outcome animation. The other sound plays.

@QThund QThund self-assigned this Dec 16, 2025
@QThund QThund requested review from a team as code owners December 16, 2025 22:43
@QThund QThund added 1-high Severely degrades primary features. Workaround may exist but user experience is very poor. shape-up It has been shaped and is awaiting the cycle. labels Dec 16, 2025
@github-actions github-actions bot requested a review from DafGreco December 16, 2025 22:44
@QThund QThund removed the request for review from mikhail-dcl December 16, 2025 22:45
… sound it was still playing

Audio clip is now null when outcome animation does not have sound.
@QThund QThund force-pushed the fix/social-emote-start-sound-keeps-playing-in-outcome branch from 05dbd15 to 7dfc809 Compare December 16, 2025 22:50
@dalkia dalkia changed the base branch from dev to fix/concurrency-issue-backpack-avatar December 18, 2025 17:14
@dalkia dalkia changed the base branch from fix/concurrency-issue-backpack-avatar to dev December 18, 2025 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-high Severely degrades primary features. Workaround may exist but user experience is very poor. shape-up It has been shaped and is awaiting the cycle.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants