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

Phone sounds update #6789

Merged
merged 4 commits into from
Aug 1, 2024
Merged

Phone sounds update #6789

merged 4 commits into from
Aug 1, 2024

Conversation

Blundir
Copy link
Contributor

@Blundir Blundir commented Jul 25, 2024

About the pull request

Adds sounds for:

Someone pick up phone on other end

Someone hung up phone on other end

Someone talk on the other end

Reaching voicemail

Dialing

Typing dial numbers

Post call disconnection beeps

(video volume is a bit low, just high it up, they are loud enough in game)

dreamseeker_yLHwKtrt8m.mp4
dreamseeker_jssosipgoJ.mp4

All sounds are either from public domain or custom made.

For discusstion about sound origin see https://discord.com/channels/150315577943130112/964684928161808384/1266875925916680253

Explain why it's good for the game

More immersion and atmosphere.

Changelog

🆑
soundadd: added numerous phone sounds for different phone interactions
/:cl:

@Blundir Blundir requested a review from fira as a code owner July 25, 2024 23:34
@Blundir Blundir marked this pull request as draft July 25, 2024 23:34
@github-actions github-actions bot added the Sound Blast 5 minutes of bass boosted music to our players label Jul 25, 2024
@Fluffyzack21
Copy link

Ngl I like this PR

@Blundir Blundir marked this pull request as ready for review July 29, 2024 14:50
Copy link
Contributor

@Drulikar Drulikar left a comment

Choose a reason for hiding this comment

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

There maybe could be better sound ending behavior (longer sounds ought to be interrupted immediately on hangup for example) but its not terrible. Feel free to revisit in the future if you care.

Also its weird that the speaker gets to_chat messages from the phone when they are speaking into the phone (as if they are hearing themselves on the line) but thats unrelated to this PR:
image

@Drulikar Drulikar added this pull request to the merge queue Aug 1, 2024
Merged via the queue into cmss13-devs:master with commit 5850a00 Aug 1, 2024
29 checks passed
cm13-github added a commit that referenced this pull request Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Sound Blast 5 minutes of bass boosted music to our players
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants