-
Notifications
You must be signed in to change notification settings - Fork 384
Conversation
@@ -572,7 +572,7 @@ IssueManagingContextControllerDelegate { | |||
case .item(let item): | |||
guard item is IssueResult else { break } | |||
updateAndScrollIfNeeded() | |||
case .list: break | |||
case .list, .clear: break |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a new case in FlatCache.Update
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm assuming this is a "correct" implementation here @rizwankce? :)
@@ -19,7 +19,7 @@ final class LocalNotificationsCache { | |||
|
|||
private let path: String | |||
private lazy var queue: FMDatabaseQueue = { | |||
return FMDatabaseQueue(path: self.path) | |||
return FMDatabaseQueue(path: self.path)! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This now returns an optional; force unwrapping for now, as it seemingly worked in the past without error. 😃
The documentation states "it returns nil
in case of an error"
@@ -9,6 +9,6 @@ Pod::Spec.new do |spec| | |||
spec.source_files = 'GitHubAPI/*.swift' | |||
spec.ios.deployment_target = '11.0' | |||
spec.watchos.deployment_target = '3.0' | |||
spec.dependency 'Alamofire', '~> 4.4.0' | |||
spec.dependency 'Alamofire', '~> 4.8' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was a bit confused that we need to update this in the podspec
and the Podfile 😅
Feels like one of the two might not be necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
⭐️ 😍
No description provided.