From fa8164cdbab7117ed1328a60cd2b1bfe364a9b98 Mon Sep 17 00:00:00 2001 From: Tobias Fella Date: Mon, 15 May 2023 21:26:29 +0200 Subject: [PATCH] Fix notifications showing up repeatedly when multiple logged-in accounts are in the same room What happens is roughly: The notification is received in one account and marked as received locally. In the other account, the event is marked as "read" in the json, which makes us delete it from the received messages. Then it is received on the first account... --- src/controller.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/src/controller.cpp b/src/controller.cpp index c077c22b8..1fc017a4e 100644 --- a/src/controller.cpp +++ b/src/controller.cpp @@ -144,7 +144,6 @@ void Controller::handleNotifications(QPointer connection) for (const auto &n : notifications) { const auto notification = n.toObject(); if (notification["read"].toBool()) { - oldNotifications.removeOne(notification["event"].toObject()["event_id"].toString()); continue; } if (oldNotifications.contains(notification["event"].toObject()["event_id"].toString())) {