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

Message notification sound #75

Closed
ugm6hr opened this issue Feb 3, 2013 · 3 comments
Closed

Message notification sound #75

ugm6hr opened this issue Feb 3, 2013 · 3 comments

Comments

@ugm6hr
Copy link

ugm6hr commented Feb 3, 2013

Nexus 7 (unrooted JB) to Galaxy Ace (rooted cyanogenmod 7) messaging on mesh (local wifi network via home router); Galaxy Ace has WhatsApp installed:
Notification on receipt of message plays last audio message sent from whatsapp - stored in "/mnt/sdcard/WhatsApp/Media/WhatsApp Audio"
Deleting the audio file returns notification to standard message tone.

@lakeman
Copy link
Member

lakeman commented Feb 4, 2013

This is a bit strange and may be a bug with WhatsApp rather than our application. We simply pass android the Notification.DEFAULT_SOUND flag.
However we should add a preference to allow the user to select from any of the installed notification sounds which should allow you to work around this issue.

@ugm6hr
Copy link
Author

ugm6hr commented Feb 4, 2013

On further testing, it does seem this is a WhatsApp bug, since it affects other notifications (not just batphone), and only when an outgoing audio message is present on WhatsApp.
I will follow up with their developer.

Date: Sun, 3 Feb 2013 16:06:55 -0800
From: notifications@github.com
To: batphone@noreply.github.com
CC: ugm6hr@hotmail.com
Subject: Re: [batphone] Message notification sound (#75)

This is a bit strange and may be a bug with WhatsApp rather than our application. We simply pass android the Notification.DEFAULT_SOUND flag.

However we should add a preference to allow the user to select from any of the installed notification sounds which should allow you to work around this issue.

          —

          Reply to this email directly or view it on GitHub.

@quixotique
Copy link
Member

Issue #86 added the new preference.

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

No branches or pull requests

3 participants