Skip to content

NotificationsManager: Group notifications per room

Aleix Pol Gonzalez requested to merge work/apol/notifications into master

Instead of issuing a new notification for every message, bundle them by room. It looks like the code was originally designed to do that and somehow we forgot along the way. It also fixes the leaking in m_notifications as we were never cleaning after it.

Marked draft because it does not feel great yet.

Edited by Aleix Pol Gonzalez

Merge request reports

Loading