Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Notifications: make them user friendly again #2130

Closed
manuroe opened this issue Apr 3, 2018 · 0 comments · Fixed by #2166
Closed

Notifications: make them user friendly again #2130

manuroe opened this issue Apr 3, 2018 · 0 comments · Fixed by #2166

Comments

@manuroe
Copy link
Member

manuroe commented Apr 3, 2018

There were a large amount of (legitimate) complaints on Goople Play and F-Droid flavours about how notifications are managed by the app since around version 0.7.5, ie, when we tried to manage android 8 restrictions enforcement on apps running in background.

The goal of this issue is to improve the UX providing more explanations to the end user because, on the codebase side, it is more a rollback in the time.

A lengthy story of the issue can be found at:
https://docs.google.com/document/d/1jy_Suwn7bFIRJA779nf6cKVlEkaOEsVTXvX9MOmYyMM/edit?usp=sharing.

This issue aims to:

  • remove sticky notifications
  • reintroduce the permission request to let the app run in background
  • give explanations why the app needs it
  • have a nice and explained fallback if the permission is not granted

Without breaking use cases listed at https://docs.google.com/spreadsheets/d/1ESuw6EUTjE9yOGNP9pCacuQdPLU1J70cvTl-TgWCHqc/edit?usp=sharing.

Here is the UX for Google play version: https://www.figma.com/proto/VKfJnrHSg5TFkGJTJUEk0rQW/Untitled?scaling=contain&node-id=82%3A566&redirected=1
No new UX on F-Droid version. Only no more sticky notifications like before.

@manuroe manuroe self-assigned this Apr 3, 2018
manuroe added a commit that referenced this issue Apr 10, 2018
…ore battery optimizations

so that it can run in background on Android M and more importantly on Android O.

#2130
manuroe added a commit that referenced this issue Apr 10, 2018
…Keep the foreground, non-removable, notification for pure background thing (initial sync, call, listen for events, ...)

#2130
manuroe added a commit that referenced this issue Apr 10, 2018
…turn false if the "run in background" permission has not been granted

#2130
manuroe added a commit that referenced this issue Apr 11, 2018
manuroe added a commit that referenced this issue Apr 11, 2018
This was referenced Apr 12, 2018
@manuroe manuroe added this to the Sprint 9 milestone Apr 19, 2018
@manuroe manuroe closed this as completed Apr 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant