[NEW] Group notifications by room

[diegolmello]

• Group notifications by room
• Show room icon
• Reply from notification

[BenWildeman]

@diegolmello I've been playing around with notifications on this repo as a test bed for my own application, and I'm wondering if grouping of notifications is the way you actually want to go?

the reason I ask, is because grouping the notification by channel would make sense only if this was like an email application, but it's not. when tapping on the notification, it will send you to the same place, regardless of what notification you tapped on, aka, the channel in which it's in regards to.

you could however group them by host, which would make more sense, and then¹ use a combination of inbox style/messaging style notifications for the rest, so you're minimising the notifications visible in the notification shade (and group). You'd only need to update the notification in which the channel is in regards to, instead of creating a whole new notification for each push sent.

Edit:
¹using a combination of notification grouping and messaging style doesn't play nicely

however I do have them grouped as per this ticket which is as follows:

[BenWildeman]

example of messaging style notification:

[diegolmello]

Very nice, @BenWildeman!
Can you open a PR with your progress?
What about notification reply? Have you played with?

[diegolmello]

Hi, @BenWildeman!
Are you still working on this?

[BenWildeman]

short answer is no

[ggazzo]

short answer is no

do you have any draft or working in progress? maybe we could continue from there ;), thx

[BenWildeman]

you know what, I don't have anything to do this weekend so I can get this sorted for you guys 🙂

[ggazzo]

really thanks ❤️