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

Batch notifications during active conversations #3872

Open
kittykat opened this issue Aug 19, 2021 · 2 comments
Open

Batch notifications during active conversations #3872

kittykat opened this issue Aug 19, 2021 · 2 comments
Labels
A-Notifications T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-GetYourUpdates Delight - Cycle 3 of Chronic issues

Comments

@kittykat
Copy link
Contributor

kittykat commented Aug 19, 2021

Describe the bug

My Android app makes my phone buzz off the desk when there is an active conversation in a room.

To Reproduce

Steps to reproduce the behavior:

  1. Lock phone
  2. Have an active conversation happen in a room with noisy notifications.
  3. Phone buzzes non-stop, draining battery a lot faster than it should (and it's very disruptive)

Expected behavior

If a user messages me or if there are multiple messages in quick succession in a room, the notifications should be batched and only the first one should be noisy. The notifications which come in quick succession after the first one should not trigger more noisy notifications and some time should pass before the next noisy notification from that chat.

Maybe notify for the first message and then not notify for the following 30 seconds if the messages are less than a second apart?

I believe Telegram batches notifications like this. Telegram notifications are broken in different and exciting ways.

See also element-hq/element-meta#360

Smartphone (please complete the following information):

  • Device: Pixel 2
  • OS: Android 11

Additional context

  • App store, version 1.1.15 [40101152] (G-b2969)
  • Homeserver: matrix.org
@kittykat kittykat added A-Notifications T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements labels Aug 19, 2021
@ouchadam ouchadam self-assigned this Oct 13, 2021
@ouchadam
Copy link
Contributor

this seems like the enhancement version of #3774

if we don't need custom timing rules we could consider this done

what do you think @kittykat ?

@kittykat
Copy link
Contributor Author

I can't reproduce right now to test because the notifications have stopped working for me altogether, but I don't think this needs to be a user configurable option. IMO, it's enough to have a sensible default.

@ouchadam ouchadam removed their assignment Sep 27, 2022
@daniellekirkwood daniellekirkwood added the Z-GetYourUpdates Delight - Cycle 3 of Chronic issues label Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Notifications T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements Z-GetYourUpdates Delight - Cycle 3 of Chronic issues
Projects
None yet
Development

No branches or pull requests

3 participants