-
Notifications
You must be signed in to change notification settings - Fork 5
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
Handle ongoing engagement in EntryWidget #1147
Handle ongoing engagement in EntryWidget #1147
Conversation
6cb5b6f
to
e01f846
Compare
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.
Left some minor suggestions and comments.
@@ -27,7 +27,8 @@ extension Glia { | |||
hasPendingInteraction: { [weak self] in | |||
guard let self else { return false } | |||
return pendingInteraction.hasPendingInteraction | |||
} | |||
}, | |||
currentInteractor: { self.interactor } |
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.
Does self
here create retain cycle?
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.
fixed 88ccb46
|
||
engagementCancellable = environment.currentInteractor()?.$currentEngagement | ||
.sink { [weak self] engagement in | ||
guard let self = self 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.
guard let self = self else { return } | |
guard let self 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.
fixed 88ccb46
engagementCancellable = environment.currentInteractor()?.$currentEngagement | ||
.sink { [weak self] engagement in | ||
guard let self = self else { return } | ||
self.ongoingEngagement = engagement |
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.
self.ongoingEngagement = engagement | |
ongoingEngagement = engagement |
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.
fixed 88ccb46
76c87b1
to
c57e113
Compare
1648c85
to
65db29f
Compare
!squash |
This PR handles ongoing engagement in EntryWidget for both core engagements and call visualizer. MOB-3806
65db29f
to
29ceec9
Compare
c735b3e
into
feature/entry-widget-and-secure-conversations-v2
What was solved?
This PR handles ongoing engagement in EntryWidget for both core engagements and call visualizer.
MOB-3806
Release notes:
Additional info:
Screenshots: