Move the firstRooms signpost out of the RoomSummaryProvider so that it doesn't invoked for each instance

This commit is contained in:
Stefan Ceriu
2023-08-28 10:58:59 +03:00
committed by Stefan Ceriu
parent 569620c52f
commit 5162d0c726
2 changed files with 2 additions and 2 deletions
@@ -400,6 +400,8 @@ class ClientProxy: ClientProxyProtocol {
.finish() .finish()
let roomListService = syncService.roomListService() let roomListService = syncService.roomListService()
ServiceLocator.shared.analytics.signpost.beginFirstRooms()
let eventStringBuilder = RoomEventStringBuilder(stateEventStringBuilder: RoomStateEventStringBuilder(userID: userID)) let eventStringBuilder = RoomEventStringBuilder(stateEventStringBuilder: RoomStateEventStringBuilder(userID: userID))
roomSummaryProvider = RoomSummaryProvider(roomListService: roomListService, roomSummaryProvider = RoomSummaryProvider(roomListService: roomListService,
eventStringBuilder: eventStringBuilder, eventStringBuilder: eventStringBuilder,
@@ -66,8 +66,6 @@ class RoomSummaryProvider: RoomSummaryProviderProtocol {
self.appSettings = appSettings self.appSettings = appSettings
self.backgroundTaskService = backgroundTaskService self.backgroundTaskService = backgroundTaskService
ServiceLocator.shared.analytics.signpost.beginFirstRooms()
diffsPublisher diffsPublisher
.receive(on: serialDispatchQueue) .receive(on: serialDispatchQueue)
.sink { [weak self] in self?.updateRoomsWithDiffs($0) } .sink { [weak self] in self?.updateRoomsWithDiffs($0) }