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

Android: silent channel using playSound flag #1736

Merged
merged 2 commits into from
Nov 10, 2020
Merged

Conversation

halaei
Copy link
Contributor

@halaei halaei commented Nov 8, 2020

Currently it isn't possible to make silent channels, and sending silent notifications to Android > 8 devices.
A playSound option is added to PushNotification.createChannel() to make it possible to create silent channels in Android. The same option exists in PushNotification.localNotification().

Related issues: #1432 and #1045

@Dallas62
Copy link
Collaborator

Dallas62 commented Nov 9, 2020

Hi,
What about setting the sound as an empty string ?
Regards

@halaei
Copy link
Contributor Author

halaei commented Nov 9, 2020

🤷🏻‍♂️ Empty string probably works fine. If backward compatibility isn't a concern, I would like to use anything like null/undefined/false/"silente"/"none" too. I am not sure which option is more relatable.
As I mentioned, I thought maybe because there already is a playSound option in the other place, it's OK to have it here too.

@Dallas62 Dallas62 changed the base branch from master to dev November 9, 2020 18:53
@Dallas62
Copy link
Collaborator

Dallas62 commented Nov 9, 2020

👍
Is it possible to reflect the change in the README ?

@halaei
Copy link
Contributor Author

halaei commented Nov 9, 2020

Sure. In a day.

@Dallas62 Dallas62 merged commit e370aeb into zo0r:dev Nov 10, 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.

None yet

2 participants