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

fix local playsounds falloff #225

Merged
merged 1 commit into from
Sep 18, 2021

Conversation

Cenrus
Copy link
Contributor

@Cenrus Cenrus commented Sep 2, 2021

About The Pull Request

Fixes local_playsound falloff. Instruments didn't specify a max_distance for the sound to play so it defaults to zero and doesn't play unless you're on the exact turf as the instrument

Why It's Good For The Game

featuring bugs from the WhiteSands codebase
i must listen to among us trap remix

Changelog

🆑
fix: Fixed musical instruments only playing sound on the turf they're in
/:cl:

@keywordlabeler keywordlabeler bot added the Fix Causes more bugs in the future that we won't know about. label Sep 2, 2021
Copy link
Member

@ZephyrTFA ZephyrTFA left a comment

Choose a reason for hiding this comment

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

Instead of changing the behavior for not specifying a distance, have instruments actually specify a distance

@shiptest-server
Copy link
Member

Test Merge Deployed

Server Instance

shiptest

Revision

Origin: 033ea7d
Pull Request: 5b60401
Server: dd05c1250d57bfccc2a44d8462a4c502f044d617

@shiptest-server
Copy link
Member

Test Merge Updated

Server Instance

shiptest

Revision

Origin: 033ea7d
Pull Request: 5b60401
Server: 7c123be6a9d9ffcf192aa21466691c922a728b54

@shiptest-server
Copy link
Member

Test Merge Updated

Server Instance

shiptest

Revision

Origin: 9693b43
Pull Request: 5b60401
Server: 651bb7bd5250da841cf57899f6ff939ae8b47202

@shiptest-server
Copy link
Member

Test Merge Updated

Server Instance

shiptest

Revision

Origin: ca83a63
Pull Request: 5b60401
Server: b9679cc28f47ef4054511559ac3e5f35dd3100bb

@shiptest-server
Copy link
Member

Test Merge Updated

Server Instance

shiptest

Revision

Origin: ca83a63
Pull Request: 5b60401
Server: 52f18b307da9ed9bafb2f173c5db9806fde13a70

@github-actions
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 14 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale All falls to entropy eventually. label Sep 16, 2021
@MarkSuckerberg MarkSuckerberg merged commit aa44843 into shiptest-ss13:master Sep 18, 2021
@Cenrus Cenrus deleted the playsounds-fix branch October 1, 2021 12:23
@ZephyrTFA ZephyrTFA removed the Stale All falls to entropy eventually. label Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Causes more bugs in the future that we won't know about.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants