element-hq / element-hq/element-android
Disabling notifications also make incoming call not working
- Dominant language
- Kotlin
- Stars
- 3.7k
- Forks
- 917
- PR merge metrics
- No merged PRs in 30d
Description
## Next steps:
The decision has been made to keep the behaviour the same but update the copy so that users understand the impact of disabling notifications this way.
Design will need to provide suggested copy and the desired delivery of such method for discussion with the team (sub-text, pop-up, etc)
## Original Issue:
**The problem is only observed when the application is on background, which I assume is the case most of the time**
When user disables the notifications, at account level or at session level from the settings of the app:

It's not mentioned that a side effect is that no sync will be performed anymore when the app is in background and so the application cannot be aware of any incoming calls.
## Possible quick fix
We should modify the wording of the settings to warn the user about this (maybe but probably) unexpected side effect.
## Possible fix 1
When user disable the notification the app will not remove the pusher, but will modify the push rule to not filter out the incoming call event (`m.call.invite`). It will not work for encrypted rooms, so we will receive a push for all events from e2e.
## Possible fix 2
Have a special sync thread to handle call signaling, which is always running, even if notification are disable (same pb for e2e rooms)
## Possible fix 3
When unchecked, the setting "Enable notifications for this session" will not remove the pusher, but just make the app not display any notifications for messages at all (so the app will only display incoming call notification).
We could add another setting to fully remove the pusher, and which explicitly warn the user that incoming call will not work when the app is in background.
## Other possible fix?
Contributor guide
Assessment
This issue has not been assessed yet.