search

jroal / a2dpvolume

Automatically exported from code.google.com/p/a2dpvolume
http://jimroal.com/slist.htm
95 stars 33 forks source link

No way to disable notifications anymore #269

Closed uschindler closed 5 years ago

uschindler commented 5 years ago

With the latest Update, regardless of the setting to show/not show notifications on connect/disconnect, it still shows them always. There is no way to get rid of them from inside A2DP volume.

The only way to get rid of notifications is by disabling them completely for the A2DP volume app using the Android notification settings.

jroal commented 5 years ago

In Android 8.0 and up notifications can be managed to the users liking through the OS. I need to just remove the setting actually. There are now 2 "notification channels" and the user can configure how they work using the settings in the OS. What I probably need to do is make a handy shortcut to open the OS notification settings for the app.

This app is starting to take too much work. I am seriously considering just abandoning it as I don't have the time to keep it up. . If you have some good ideas on how to make it more useful, I would happily take implemented code. I will keep this issue on the list though and if/when I get some spare time I can look into it. Thanks for posting.
uschindler commented 5 years ago

Ah okay. So the setting is just a relic. It's no problem, as I disabled notifications through Android already. Thanks.

jroal commented 5 years ago

Keep in mind that Android may stop apps that are running in the background without an active notification.

By the way, I was able to configure the foreground channel to show the icon and the background channel to only show when you slide down from the top. This is behavior people asked for years ago and now you can do it. The background notification is enough to keep Android from stopping the app.

uschindler commented 5 years ago

Yeah, I know this. But it's easy to prevent this with recent Android. You can put apps on a list of app that should not be killed. See other issue #268. Works perfectly, notifications are not needed.

jroal commented 5 years ago

Here is the code in Service.java where the setting is overridden for API 25 and up.

// we will ignore the no notifications preference in API 25 and up since this can be managed
        // by the user now within Android
        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N_MR1) notify = true;
jroal commented 5 years ago

Closing this since you can manage notifications in Android now.