From 01dcfef87eaaedd720633ddfab6eafa5cce6ad46 Mon Sep 17 00:00:00 2001 From: George Nachman Date: Fri, 11 Mar 2022 13:50:23 -0800 Subject: [PATCH] Remove unneeded todo Bulk profile fetching is pretty complicated. It already limits its impact by throttling the rate at which it makes requests. I don't think it's worth trying to make it do batch operations at this time, so I'm removing this TODO. This stacks on top of PR #4068 --- SignalMessaging/contacts/OWSContactsManager.swift | 1 - 1 file changed, 1 deletion(-) diff --git a/SignalMessaging/contacts/OWSContactsManager.swift b/SignalMessaging/contacts/OWSContactsManager.swift index 9c04440fb6..9ef2421e96 100644 --- a/SignalMessaging/contacts/OWSContactsManager.swift +++ b/SignalMessaging/contacts/OWSContactsManager.swift @@ -1005,7 +1005,6 @@ extension OWSContactsManager { return CommonFormats.formatUsername(username) } }.refine { addresses in - // TODO: Combine these db queries into one. return addresses.lazy.map { self.fetchProfile(forUnknownAddress: $0) return self.unknownUserLabel