mirror of
https://github.com/Novaloop-AG/element-x-ios.git
synced 2026-11-05 20:09:04 +00:00
vector-im/element-meta/issues/2206 - Fix room avatars not showing up on the message forwarding screen
This commit is contained in:
committed by
Stefan Ceriu
parent
9a52dcf9ac
commit
4447829618
@@ -749,6 +749,7 @@ class RoomFlowCoordinator: FlowCoordinatorProtocol {
|
|||||||
let messageForwardingNavigationStackCoordinator = NavigationStackCoordinator()
|
let messageForwardingNavigationStackCoordinator = NavigationStackCoordinator()
|
||||||
|
|
||||||
let parameters = MessageForwardingScreenCoordinatorParameters(roomSummaryProvider: roomSummaryProvider,
|
let parameters = MessageForwardingScreenCoordinatorParameters(roomSummaryProvider: roomSummaryProvider,
|
||||||
|
mediaProvider: userSession.mediaProvider,
|
||||||
sourceRoomID: roomProxy.id)
|
sourceRoomID: roomProxy.id)
|
||||||
let coordinator = MessageForwardingScreenCoordinator(parameters: parameters)
|
let coordinator = MessageForwardingScreenCoordinator(parameters: parameters)
|
||||||
|
|
||||||
|
|||||||
+2
@@ -19,6 +19,7 @@ import SwiftUI
|
|||||||
|
|
||||||
struct MessageForwardingScreenCoordinatorParameters {
|
struct MessageForwardingScreenCoordinatorParameters {
|
||||||
let roomSummaryProvider: RoomSummaryProviderProtocol
|
let roomSummaryProvider: RoomSummaryProviderProtocol
|
||||||
|
let mediaProvider: MediaProviderProtocol
|
||||||
let sourceRoomID: String
|
let sourceRoomID: String
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -41,6 +42,7 @@ final class MessageForwardingScreenCoordinator: CoordinatorProtocol {
|
|||||||
self.parameters = parameters
|
self.parameters = parameters
|
||||||
|
|
||||||
viewModel = MessageForwardingScreenViewModel(roomSummaryProvider: parameters.roomSummaryProvider,
|
viewModel = MessageForwardingScreenViewModel(roomSummaryProvider: parameters.roomSummaryProvider,
|
||||||
|
mediaProvider: parameters.mediaProvider,
|
||||||
sourceRoomID: parameters.sourceRoomID)
|
sourceRoomID: parameters.sourceRoomID)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
+2
-1
@@ -30,11 +30,12 @@ class MessageForwardingScreenViewModel: MessageForwardingScreenViewModelType, Me
|
|||||||
}
|
}
|
||||||
|
|
||||||
init(roomSummaryProvider: RoomSummaryProviderProtocol,
|
init(roomSummaryProvider: RoomSummaryProviderProtocol,
|
||||||
|
mediaProvider: MediaProviderProtocol,
|
||||||
sourceRoomID: String) {
|
sourceRoomID: String) {
|
||||||
self.roomSummaryProvider = roomSummaryProvider
|
self.roomSummaryProvider = roomSummaryProvider
|
||||||
self.sourceRoomID = sourceRoomID
|
self.sourceRoomID = sourceRoomID
|
||||||
|
|
||||||
super.init(initialViewState: MessageForwardingScreenViewState())
|
super.init(initialViewState: MessageForwardingScreenViewState(), imageProvider: mediaProvider)
|
||||||
|
|
||||||
roomSummaryProvider.roomListPublisher
|
roomSummaryProvider.roomListPublisher
|
||||||
.receive(on: DispatchQueue.main)
|
.receive(on: DispatchQueue.main)
|
||||||
|
|||||||
@@ -108,7 +108,9 @@ private struct MessageForwardingRoomCell: View {
|
|||||||
struct MessageForwardingScreen_Previews: PreviewProvider, TestablePreview {
|
struct MessageForwardingScreen_Previews: PreviewProvider, TestablePreview {
|
||||||
static var previews: some View {
|
static var previews: some View {
|
||||||
let summaryProvider = MockRoomSummaryProvider(state: .loaded(.mockRooms))
|
let summaryProvider = MockRoomSummaryProvider(state: .loaded(.mockRooms))
|
||||||
let viewModel = MessageForwardingScreenViewModel(roomSummaryProvider: summaryProvider, sourceRoomID: "")
|
let viewModel = MessageForwardingScreenViewModel(roomSummaryProvider: summaryProvider,
|
||||||
|
mediaProvider: MockMediaProvider(),
|
||||||
|
sourceRoomID: "")
|
||||||
|
|
||||||
NavigationStack {
|
NavigationStack {
|
||||||
MessageForwardingScreen(context: viewModel.context)
|
MessageForwardingScreen(context: viewModel.context)
|
||||||
|
|||||||
@@ -27,7 +27,9 @@ class MessageForwardingScreenViewModelTests: XCTestCase {
|
|||||||
|
|
||||||
override func setUpWithError() throws {
|
override func setUpWithError() throws {
|
||||||
cancellables.removeAll()
|
cancellables.removeAll()
|
||||||
viewModel = MessageForwardingScreenViewModel(roomSummaryProvider: MockRoomSummaryProvider(state: .loaded(.mockRooms)), sourceRoomID: "1")
|
viewModel = MessageForwardingScreenViewModel(roomSummaryProvider: MockRoomSummaryProvider(state: .loaded(.mockRooms)),
|
||||||
|
mediaProvider: MockMediaProvider(),
|
||||||
|
sourceRoomID: "1")
|
||||||
context = viewModel.context
|
context = viewModel.context
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -1,3 +1,3 @@
|
|||||||
version https://git-lfs.github.com/spec/v1
|
version https://git-lfs.github.com/spec/v1
|
||||||
oid sha256:0df3d63b8e54f9d6bef4515a687e319e06324ead88e17ef0e9522a426f6fd7fe
|
oid sha256:afbc0d42973b4360949b551b5e7d34543a262eef3ea29c3bc1219c06d2249f5a
|
||||||
size 158624
|
size 170537
|
||||||
|
|||||||
@@ -0,0 +1 @@
|
|||||||
|
Fix room avatars not showing up on the message forwarding screen
|
||||||
Reference in New Issue
Block a user