From 64bcc9458921852bb37581f7182a1c33ab9b4de6 Mon Sep 17 00:00:00 2001 From: Michael Kirk Date: Mon, 8 May 2017 14:46:35 -0400 Subject: [PATCH] Instead of alert we're providing in context reminders - no need for these TODOs // FREEBIE --- Signal/src/contact/SystemContactsFetcher.swift | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/Signal/src/contact/SystemContactsFetcher.swift b/Signal/src/contact/SystemContactsFetcher.swift index 79cc2ee8f..efd70531d 100644 --- a/Signal/src/contact/SystemContactsFetcher.swift +++ b/Signal/src/contact/SystemContactsFetcher.swift @@ -58,15 +58,12 @@ class SystemContactsFetcher: NSObject { } if !granted { - // TODO, make this a one time dismissable admonishment - // e.g. remember across launches that the user has dismissed. - self.displayMissingContactsPermissionAlert() + Logger.info("\(self.TAG) declined contact access.") } else { self.updateContacts() } }) case .authorized: - // TODO reset onetime admonishment reminder, so that we remind user again (once) if they've since toggled permissions. self.updateContacts() case .denied, .restricted: Logger.debug("\(TAG) contacts were \(self.authorizationStatus)") @@ -82,11 +79,6 @@ class SystemContactsFetcher: NSObject { updateContacts() } - private func displayMissingContactsPermissionAlert() { - let foo = UIApplication.shared.frontmostViewController - Logger.error("TODO") - } - private func updateContacts() { AssertIsOnMainThread()