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.

Sign up for GitHub

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

Jump to bottom

Push notifications are not grouped #994

Closed
kongo09 opened this issue Jul 27, 2023 · 1 comment · Fixed by #1666
Closed

Push notifications are not grouped #994

kongo09 opened this issue Jul 27, 2023 · 1 comment · Fixed by #1666
Assignees
@bmarty
Labels
A-Notifications O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@kongo09
Copy link

kongo09 commented Jul 27, 2023

Steps to reproduce

  1. Receive more than one push notification

Outcome

What did you expect?

Notifications are grouped together

What happened instead?

Screenshot_20230727-235050

Your phone model

Pixel 5

Operating system version

Android 13

Application version and app store

latest nightly EAX

Homeserver

matrix.org

Will you send logs?

No

Are you willing to provide a PR?

No
@kongo09 kongo09 added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Jul 27, 2023
@kongo09
Copy link
Author

kongo09 commented Jul 29, 2023

Beyond grouping, I would also expect that only the most recent unread message from the same room is shown. These notifications here all refer to the same sender in the same room

screen-20230729-183103.mp4

@Florian14 Florian14 added S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-Notifications labels Aug 4, 2023
@bmarty bmarty self-assigned this Oct 26, 2023
@bmarty bmarty mentioned this issue Oct 27, 2023
Merged
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bmarty bmarty

Labels
A-Notifications O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Milestone
No milestone
4 participants
@kittykat @kongo09 @bmarty @Florian14