search

citiususc / calendula

An Android assistant for personal medication management
https://citius.usc.es/calendula/
GNU General Public License v3.0
208 stars 107 forks source link

Add Notifications management settings (API 26+) #156

Closed aoudiamoncef closed 6 years ago

aoudiamoncef commented 6 years ago

Open system notifications setting of Calendula from Notification settings.

AlvaroBrey commented 6 years ago

This also raises an interesting question: we're hiding the notification channels on builds under O, but we're not hiding the old way on O and upper. We could probably hide the ringtone and vibration prefs on builds over O, since they are managed via the channels.

That's for another PR though.

aoudiamoncef commented 6 years ago

This also raises an interesting question: we're hiding the notification channels on builds under O, but we're not hiding the old way on O and upper. We could probably hide the ringtone and vibration prefs on builds over O, since they are managed via the channels.

That's for another PR though.

I implemented this behavior in my new commit :)

AlvaroBrey commented 6 years ago

Looks good, thank you for your contribution!