Skip to content

Conversation

BillCarsonFr
Copy link
Member

Pull Request Checklist

Fixes Fixes #4493
Based on #4499

UI changes have been tested with:

  • iPhone and iPad simulators in portrait and landscape orientations.
  • Dark mode enabled and disabled.
  • Various sizes of dynamic type.
  • Voiceover enabled.

Copy link

codecov bot commented Sep 12, 2025

Codecov Report

❌ Patch coverage is 0% with 38 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.16%. Comparing base (3fa133c) to head (cae1e62).
⚠️ Report is 1 commits behind head on develop.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...rces/Services/ElementCall/ElementCallService.swift 0.00% 37 Missing ⚠️
ElementX/Sources/Other/SDKListener.swift 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4505      +/-   ##
===========================================
- Coverage    79.18%   79.16%   -0.03%     
===========================================
  Files          836      836              
  Lines        78731    78758      +27     
===========================================
- Hits         62347    62346       -1     
- Misses       16384    16412      +28     
Flag Coverage Δ
unittests 71.51% <0.00%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@BillCarsonFr BillCarsonFr force-pushed the valere/call/decline_ringing branch from 60c329d to b480695 Compare September 16, 2025 07:38
@BillCarsonFr BillCarsonFr force-pushed the valere/call/subscribe_to_call_decline branch from 1c8addd to 12fd7e1 Compare September 16, 2025 08:04
Base automatically changed from valere/call/decline_ringing to develop September 16, 2025 08:14
@BillCarsonFr BillCarsonFr force-pushed the valere/call/subscribe_to_call_decline branch from 3509092 to f847108 Compare September 16, 2025 08:29
@BillCarsonFr BillCarsonFr added the pr-feature for a new feature label Sep 16, 2025
@BillCarsonFr BillCarsonFr marked this pull request as ready for review September 16, 2025 08:56
@BillCarsonFr BillCarsonFr requested a review from a team as a code owner September 16, 2025 08:56
@BillCarsonFr BillCarsonFr requested review from pixlwave and removed request for a team September 16, 2025 08:56
Copy link
Member

@pixlwave pixlwave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, looks like a neat feature 🙌

I chatted with Stefan about the new DeclineCallbackPublisher and we concluded that as there is only a single usage, the additional complexity probably isn't worth it. We're perfectly happy to follow the same pattern for these as we do for e.g. knocking, typing etc by subscribing and exposing a single publisher:

// periphery:ignore - required for instance retention in the rust codebase
private var typingNotificationObservationToken: TaskHandle?

private let typingMembersSubject = CurrentValueSubject<[String], Never>([])
var typingMembersPublisher: CurrentValuePublisher<[String], Never> {
typingMembersSubject.asCurrentValuePublisher()
}

Task {
subscribeToRoomInfoUpdates()
subscribeToTypingNotifications()
await subscribeToKnockRequests()
if infoPublisher.value.isEncrypted {
await subscribeToIdentityStatusChanges()
}
}

private func subscribeToTypingNotifications() {
typingNotificationObservationToken = room.subscribeToTypingNotifications(listener: RoomTypingNotificationUpdateListener { [weak self] typingUserIDs in
guard let self else { return }
MXLog.info("Received typing notification update, typingUsers: \(typingUserIDs)")
let typingMembers = typingUserIDs.compactMap { userID in
if let member = self.membersPublisher.value.filter({ $0.userID == userID }).first {
return member.displayName ?? member.userID
} else {
return userID
}
}
typingMembersSubject.send(typingMembers)
})
}

@BillCarsonFr BillCarsonFr force-pushed the valere/call/subscribe_to_call_decline branch from 48c1f7a to 2c242a3 Compare September 18, 2025 08:17
Copy link
Member

@pixlwave pixlwave left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the updates.

Copy link

@BillCarsonFr BillCarsonFr merged commit 2eaad89 into develop Sep 18, 2025
13 checks passed
@BillCarsonFr BillCarsonFr deleted the valere/call/subscribe_to_call_decline branch September 18, 2025 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-feature for a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Task] Element-Call Send RTC decline when the Incoming Call notification decline button is called
2 participants