-
Notifications
You must be signed in to change notification settings - Fork 212
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
[CIS-932] Fix composer overlapping last message #1210
Conversation
@@ -482,6 +482,8 @@ open class _ChatMessageListVC<ExtraData: ExtraDataTypes>: | |||
_ channelController: _ChatChannelController<ExtraData>, | |||
didChangeTypingMembers typingMembers: Set<_ChatChannelMember<ExtraData.User>> | |||
) { | |||
guard channelController.areTypingEventsEnabled else { return } |
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.
Awesome work 🎉
04d9b86
to
4507e1f
Compare
Codecov Report
@@ Coverage Diff @@
## main #1210 +/- ##
=======================================
Coverage 91.25% 91.25%
=======================================
Files 217 217
Lines 9277 9277
=======================================
Hits 8466 8466
Misses 811 811
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
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! Nice Catch ✅
4507e1f
to
da6a8d8
Compare
This happened for channels with typing events disabled