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

Add preference to mute call ringtone #1273

Merged
merged 5 commits into from
Mar 6, 2020
Merged

Add preference to mute call ringtone #1273

merged 5 commits into from
Mar 6, 2020

Conversation

LuKks
Copy link

@LuKks LuKks commented Feb 7, 2020

fix #209


IssueHunt Summary

Referenced issues

This pull request has been submitted to:


IssueHunt has been backed by the following sponsors. Become a sponsor

@LuKks LuKks requested a review from sindresorhus February 10, 2020 20:28
source/index.ts Outdated Show resolved Hide resolved
source/menu.ts Outdated Show resolved Hide resolved
source/index.ts Outdated Show resolved Hide resolved
@sindresorhus
Copy link
Owner

Have you manually tested that this actually works, by receiving a call and ensuring it doesn't make a sound?

@LuKks
Copy link
Author

LuKks commented Mar 3, 2020

Yes. This is what I commented in the issue discussion some time ago:

Due HTTP cache, for example, if the ringtone plays and now you disable it, the sound is already cached so you will need to restart the app.

I hope that is ok and I also tested it again, it works fine. By the way, really thank you for the corrections in the code.

@sindresorhus sindresorhus changed the title Call ringtone muted Add preference to mute call ringtone Mar 6, 2020
@sindresorhus sindresorhus merged commit 5daa934 into sindresorhus:master Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Call sound should also be disabled
2 participants