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

Improved Adminhelp sound #26319

Closed
wants to merge 3 commits into from
Closed

Improved Adminhelp sound #26319

wants to merge 3 commits into from

Conversation

VloBoo
Copy link

@VloBoo VloBoo commented Mar 21, 2024

About the PR

Replaced the adminhelp sound with a more attractive sound of a falling pipe

Why / Balance

During the chaos around April 1, people may not notice that an admin is writing to them. Therefore, I believe that the sound needs to be improved so that everyone can hear it clearly.

Technical details

I hope I didn't make any stupid mistakes in the PR.

Media

sus.mp4
  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog
🆑

  • tweak: Improved Adminhelp sound!

Comment on lines +219 to +223

- files: ["metal-pipe-falling.ogg"]
copyright: "Taken from thenotcheeseman via freesound.org and mixed from stereo to mono."
license: CC0-1.0
source: "https://freesound.org/people/thenotcheeseman/sounds/679206/"
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm reasonably sure this is not licensed properly and is a stolen sound effect from the original pack from the 90s.

Copy link
Author

Choose a reason for hiding this comment

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

Unfortunately, I could not find another original. (>︿<)
Is this source definitely not suitable as the original?

@CatMagic1
Copy link

new player experience improved by 110% on april fools day

@FairlySadPanda
Copy link
Contributor

Sloth please just this once

please

@CatMagic1
Copy link

i too will beg if necessary

@github-actions github-actions bot added the Merge Conflict This PR currently has conflicts that need to be addressed. label Mar 25, 2024
Copy link
Contributor

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

@github-actions github-actions bot removed the Merge Conflict This PR currently has conflicts that need to be addressed. label Mar 27, 2024
@MaloTV
Copy link
Contributor

MaloTV commented Mar 28, 2024

Oh god please i require this

@github-actions github-actions bot added the Merge Conflict This PR currently has conflicts that need to be addressed. label Mar 31, 2024
Copy link
Contributor

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

@Slava0135
Copy link
Member

as i figured out in #26078 metal pipe sound can't be used

@Slava0135 Slava0135 closed this Mar 31, 2024
@VloBoo VloBoo deleted the my-ears branch March 31, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
April Fools (merge for one day) Merge Conflict This PR currently has conflicts that need to be addressed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants