Instead of alert we're providing in context reminders - no need for

these TODOs

// FREEBIE
pull/1/head
Michael Kirk 8 years ago
parent 04878bf222
commit 64bcc94589

@ -58,15 +58,12 @@ class SystemContactsFetcher: NSObject {
} }
if !granted { if !granted {
// TODO, make this a one time dismissable admonishment Logger.info("\(self.TAG) declined contact access.")
// e.g. remember across launches that the user has dismissed.
self.displayMissingContactsPermissionAlert()
} else { } else {
self.updateContacts() self.updateContacts()
} }
}) })
case .authorized: case .authorized:
// TODO reset onetime admonishment reminder, so that we remind user again (once) if they've since toggled permissions.
self.updateContacts() self.updateContacts()
case .denied, .restricted: case .denied, .restricted:
Logger.debug("\(TAG) contacts were \(self.authorizationStatus)") Logger.debug("\(TAG) contacts were \(self.authorizationStatus)")
@ -82,11 +79,6 @@ class SystemContactsFetcher: NSObject {
updateContacts() updateContacts()
} }
private func displayMissingContactsPermissionAlert() {
let foo = UIApplication.shared.frontmostViewController
Logger.error("TODO")
}
private func updateContacts() { private func updateContacts() {
AssertIsOnMainThread() AssertIsOnMainThread()

Loading…
Cancel
Save