Fix usage of combine framework

The sinks have to run in the main queue because they are triggering ui updates (by defaults they run in the queue the nsnotification was sent in).
monal-im · Sep 14, 2024 · 7e8ced1 · 7e8ced1
1 parent bd4e00f
commit 7e8ced1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/Monal/Classes/ContactsView.swift b/Monal/Classes/ContactsView.swift
@@ -184,8 +184,10 @@ class Contacts: ObservableObject {
         self.requestCount = DataLayer.sharedInstance().allContactRequests().count
         subscriptions = [
             NotificationCenter.default.publisher(for: NSNotification.Name("kMonalContactRemoved"))
+                .receive(on: DispatchQueue.main)
                 .sink() { _ in self.refreshContacts() },
             NotificationCenter.default.publisher(for: NSNotification.Name("kMonalContactRefresh"))
+                .receive(on: DispatchQueue.main)
                 .sink() { _ in self.refreshContacts() }
         ]
     }

diff --git a/Monal/Classes/OmemoKeysView.swift b/Monal/Classes/OmemoKeysView.swift
@@ -454,8 +454,10 @@ class OmemoKeysForChat: ObservableObject {
         self.contacts = OmemoKeysForChat.knownDevices(viewContact: self.viewContact)
         subscriptions = [
             NotificationCenter.default.publisher(for: NSNotification.Name("kMonalOmemoStateUpdated"))
+                .receive(on: DispatchQueue.main)
                 .sink() { _ in self.updateContactDevices() },
             NotificationCenter.default.publisher(for: NSNotification.Name("kMonalMucParticipantsAndMembersUpdated"))
+                ,receive(on: DispatchQueue.main)
                 .sink() { _ in self.updateContactDevices() },
         ]
     }