-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge upstream 3.4.2 #24
Conversation
Fix compilation issues by importing UIKit (dev branch)
…odspec, update IBAV
iOS 14, XCode 12, Swift 5.3, goodbye iOS 11
Prepare release 3.4.0
…ssage Fix casting MessageType to custom type for LinkPreview cells
Remove keyboard observers in viewWillDisappear, not deinit
======= | ||
PODFILE CHECKSUM: 24c39a08ead4a8771aa94310cfc35fe3be6b3105 | ||
>>>>>>> 3.4.2 | ||
PODFILE CHECKSUM: 31c6eb1d734807f78336ea60201a116e2410b2dd |
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 checksum was recalculated by bundle exec pod install
@@ -29,7 +29,7 @@ internal extension MessageKind { | |||
switch self { | |||
case .linkPreview(let linkItem): | |||
return linkItem.textKind | |||
case .text, .emoji, .attributedText: | |||
case .announcement, .text, .emoji, .attributedText: |
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.
.announcement
was added by us.
@@ -7,6 +7,7 @@ | |||
// | |||
|
|||
import Foundation | |||
import UIKit |
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.
From some version of Swift Package Manager, packages are not be compiled without declaring the correct imported modules.
In this file UIFont
, UIEdgeInsets
and other UI
-prefixed classes are used.
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.
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.
LGTM 👍
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.
LGTM
Overview
import UIKit
at 304483d3a2ac0f and 304483d need to be reviewed.
Screenshot