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

bowls now make drinking sounds #32819

Merged
merged 1 commit into from
Oct 26, 2024

Conversation

Moomoobeef
Copy link
Contributor

About the PR

Bowls, which can be filled with fluid, now make a drinking sound when drank from. No longer can you eat water.
Conveniently they still make an eating sound when filled with food (IE used in cooking recipe) so that's a plus as well.

Why / Balance

Fixes #32224

Technical details

Media

Requirements

Breaking changes

Changelog
🆑

  • fix: Bowls no longer make an eating sound when drinking from them.

@github-actions github-actions bot added the Changes: No C# Changes: Requires no C# knowledge to review or fix this item. label Oct 14, 2024
@Cojoke-dot
Copy link
Contributor

O7 crunchy water, hello slurpy rice and pork

@Moomoobeef
Copy link
Contributor Author

Moomoobeef commented Oct 14, 2024

O7 crunchy water, hello slurpy rice and pork

in my testing rice still made eating sounds. Didn't test pork but it's probably the same.
I didn't actually look into why but I would assume it's because the bowls of food are parented to basefood and the bowl? Or they just have the food sound defined in them for redundancy? Either way, bowls of food have their own prototypes. Like I said, i didn't look :P but it is convenient.

@slarticodefast slarticodefast self-assigned this Oct 16, 2024
@slarticodefast slarticodefast added the S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval. label Oct 16, 2024
@slarticodefast
Copy link
Member

Thanks for the fix!

@chromiumboy
Copy link
Contributor

I tested it and seems like soups and other water based dishes still have eating sounds. But pure liquids have slurping as intended

@chromiumboy
Copy link
Contributor

This doesn't fix all the issues, but its a step in the right direction

@chromiumboy chromiumboy merged commit 97ef463 into space-wizards:master Oct 26, 2024
12 checks passed
GitHubUser53123 pushed a commit to RonRonstation/ronstation that referenced this pull request Oct 26, 2024
* Add flash reaction effect (space-wizards#32377)

add flash reaction effect

* Automatic changelog update

* Fix loneop spawnrate by reverting it to not use the shuttle event system. (space-wizards#32942)

Fix loneop spawnrate by reverting it to not use the custom shuttle event system.

* Automatic changelog update

* add atmosia to devmap (space-wizards#32460)

change

* bowls now make drinking sounds (space-wizards#32819)

* Automatic changelog update

---------

Co-authored-by: slarticodefast <161409025+slarticodefast@users.noreply.github.com>
Co-authored-by: PJBot <pieterjan.briers+bot@gmail.com>
Co-authored-by: BramvanZijp <56019239+BramvanZijp@users.noreply.github.com>
Co-authored-by: Ilya246 <57039557+Ilya246@users.noreply.github.com>
Co-authored-by: Moomoobeef <62638182+Moomoobeef@users.noreply.github.com>
GitHubUser53123 pushed a commit to RonRonstation/ronstation that referenced this pull request Oct 27, 2024
* Add flash reaction effect (space-wizards#32377)

add flash reaction effect

* Automatic changelog update

* Fix loneop spawnrate by reverting it to not use the shuttle event system. (space-wizards#32942)

Fix loneop spawnrate by reverting it to not use the custom shuttle event system.

* Automatic changelog update

* add atmosia to devmap (space-wizards#32460)

change

* bowls now make drinking sounds (space-wizards#32819)

* Automatic changelog update

* Add health analyzer unrevivability warning (space-wizards#32636)

* Add health analyzer unrevivability warning

* Remove errornous comment

* Automatic changelog update

* Removed the name "Hujsak" (space-wizards#32998)

removed hujsak

* Update Credits (space-wizards#33016)

Co-authored-by: PJBot <pieterjan.briers+bot@gmail.com>

* fix pie throwing sound not playing (space-wizards#33017)

* Automatic changelog update

---------

Co-authored-by: slarticodefast <161409025+slarticodefast@users.noreply.github.com>
Co-authored-by: PJBot <pieterjan.briers+bot@gmail.com>
Co-authored-by: BramvanZijp <56019239+BramvanZijp@users.noreply.github.com>
Co-authored-by: Ilya246 <57039557+Ilya246@users.noreply.github.com>
Co-authored-by: Moomoobeef <62638182+Moomoobeef@users.noreply.github.com>
Co-authored-by: Saphire Lattice <lattice@saphi.re>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
iaada pushed a commit to iaada/space-station-14 that referenced this pull request Nov 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: No C# Changes: Requires no C# knowledge to review or fix this item. S: Approved Status: Reviewed and approved by at least one maintainer; a PR may require another approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

drinking out of a bowl makes eating sound, even if it is just full of water
4 participants