Made some ChannelList parameters optional

Author: martinmitrevski

CHANGELOG.md

@@ -6,6 +6,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 ### ✅ Added
 - Add message preview with attachments in channel list
 
+### 🐞 Fixed
+- Made some `ChannelList` parameters optional
+
 # [4.39.0](https://github.com/GetStream/stream-chat-swiftui/releases/tag/4.39.0)
 _October 06, 2023_
 

Sources/StreamChatSwiftUI/ChatChannel/ChannelHeader/ChatChannelHeaderViewModifier.swift

@@ -93,7 +93,7 @@ public struct DefaultChatChannelHeader: ToolbarContent {
 
 /// The default header modifier.
 public struct DefaultChannelHeaderModifier: ChatChannelHeaderViewModifier {
-    @StateObject private var channelHeaderLoader = ChannelHeaderLoader()
+    @ObservedObject private var channelHeaderLoader = InjectedValues[\.utils].channelHeaderLoader
     @State private var isActive: Bool = false
 
     public var channel: ChatChannel

Sources/StreamChatSwiftUI/ChatChannelList/ChannelHeaderLoader.swift

@@ -36,7 +36,7 @@ open class ChannelHeaderLoader: ObservableObject {
         willSet {
             if !scheduledUpdate {
                 scheduledUpdate = true
-                DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) { [weak self] in
+                DispatchQueue.main.asyncAfter(deadline: .now() + 0.5) { [weak self] in
                     self?.objectWillChange.send()
                     self?.scheduledUpdate = false
                 }

Sources/StreamChatSwiftUI/ChatChannelList/ChatChannelList.swift

@@ -10,7 +10,7 @@ import SwiftUI
 public struct ChannelList<Factory: ViewFactory>: View {
 
     @Injected(\.colors) private var colors
-
+    
     private var factory: Factory
     var channels: LazyCachedMapCollection<ChatChannel>
     @Binding var selectedChannel: ChannelSelectionInfo?
@@ -32,24 +32,41 @@ public struct ChannelList<Factory: ViewFactory>: View {
         selectedChannel: Binding<ChannelSelectionInfo?>,
         swipedChannelId: Binding<String?>,
         scrollable: Bool = true,
-        onlineIndicatorShown: @escaping (ChatChannel) -> Bool,
-        imageLoader: @escaping (ChatChannel) -> UIImage,
+        onlineIndicatorShown: ((ChatChannel) -> Bool)? = nil,
+        imageLoader: ((ChatChannel) -> UIImage)? = nil,
         onItemTap: @escaping (ChatChannel) -> Void,
         onItemAppear: @escaping (Int) -> Void,
-        channelNaming: @escaping (ChatChannel) -> String,
+        channelNaming: ((ChatChannel) -> String)? = nil,
         channelDestination: @escaping (ChannelSelectionInfo) -> Factory.ChannelDestination,
-        trailingSwipeRightButtonTapped: @escaping (ChatChannel) -> Void,
-        trailingSwipeLeftButtonTapped: @escaping (ChatChannel) -> Void,
-        leadingSwipeButtonTapped: @escaping (ChatChannel) -> Void
+        trailingSwipeRightButtonTapped: @escaping (ChatChannel) -> Void = { _ in },
+        trailingSwipeLeftButtonTapped: @escaping (ChatChannel) -> Void = { _ in },
+        leadingSwipeButtonTapped: @escaping (ChatChannel) -> Void = { _ in }
     ) {
         self.factory = factory
         self.channels = channels
         self.onItemTap = onItemTap
         self.onItemAppear = onItemAppear
-        self.channelNaming = channelNaming
+        if let channelNaming = channelNaming {
+            self.channelNaming = channelNaming
+        } else {
+            let channelNamer = InjectedValues[\.utils].channelNamer
+            self.channelNaming = { channel in
+                channelNamer(channel, InjectedValues[\.chatClient].currentUserId) ?? ""
+            }
+        }
         self.channelDestination = channelDestination
-        self.imageLoader = imageLoader
-        self.onlineIndicatorShown = onlineIndicatorShown
+        if let imageLoader = imageLoader {
+            self.imageLoader = imageLoader
+        } else {
+            self.imageLoader = InjectedValues[\.utils].channelHeaderLoader.image(for:)
+        }
+        if let onlineIndicatorShown = onlineIndicatorShown {
+            self.onlineIndicatorShown = onlineIndicatorShown
+        } else {
+            self.onlineIndicatorShown = { channel in
+                channel.shouldShowOnlineIndicator
+            }
+        }
         self.trailingSwipeRightButtonTapped = trailingSwipeRightButtonTapped
         self.trailingSwipeLeftButtonTapped = trailingSwipeLeftButtonTapped
         self.leadingSwipeButtonTapped = leadingSwipeButtonTapped

Sources/StreamChatSwiftUI/ChatChannelList/ChatChannelListItem.swift

@@ -230,6 +230,13 @@ extension ChatChannel {
             currentUserId: InjectedValues[\.chatClient].currentUserId
         ).isEmpty && config.typingEventsEnabled
     }
+    
+    public var shouldShowOnlineIndicator: Bool {
+        !lastActiveMembers.filter { member in
+            member.isOnline && member.id != InjectedValues[\.chatClient].currentUserId
+        }
+        .isEmpty
+    }
 
     public var subtitleText: String {
         if isMuted {

Sources/StreamChatSwiftUI/ChatChannelList/ChatChannelListView.swift

@@ -176,7 +176,7 @@ public struct ChatChannelListContentView<Factory: ViewFactory>: View {
 
     private var viewFactory: Factory
     @ObservedObject private var viewModel: ChatChannelListViewModel
-    @StateObject private var channelHeaderLoader = ChannelHeaderLoader()
+    @ObservedObject private var channelHeaderLoader = InjectedValues[\.utils].channelHeaderLoader
     private var onItemTap: (ChatChannel) -> Void
 
     public init(

Sources/StreamChatSwiftUI/ChatChannelList/ChatChannelListViewModel.swift

@@ -181,10 +181,7 @@ open class ChatChannelListViewModel: ObservableObject, ChatChannelListController
     /// - Parameter channel: the provided channel.
     /// - Returns: Boolean whether the indicator is shown.
     public func onlineIndicatorShown(for channel: ChatChannel) -> Bool {
-        !channel.lastActiveMembers.filter { member in
-            member.isOnline && member.id != chatClient.currentUserId
-        }
-        .isEmpty
+        channel.shouldShowOnlineIndicator
     }
 
     public func onDeleteTapped(channel: ChatChannel) {

Sources/StreamChatSwiftUI/Utils.swift

@@ -26,6 +26,7 @@ public class Utils {
     public var snapshotCreator: SnapshotCreator
     public var messageIdBuilder: MessageIdBuilder
     public var sortReactions: (MessageReactionType, MessageReactionType) -> Bool
+    public var channelHeaderLoader: ChannelHeaderLoader
 
     var messageCachingUtils = MessageCachingUtils()
     var messageListDateUtils: MessageListDateUtils
@@ -48,6 +49,7 @@ public class Utils {
         chatUserNamer: ChatUserNamer = DefaultChatUserNamer(),
         snapshotCreator: SnapshotCreator = DefaultSnapshotCreator(),
         messageIdBuilder: MessageIdBuilder = DefaultMessageIdBuilder(),
+        channelHeaderLoader: ChannelHeaderLoader = ChannelHeaderLoader(),
         sortReactions: @escaping (MessageReactionType, MessageReactionType) -> Bool = Utils.defaultSortReactions,
         shouldSyncChannelControllerOnAppear: @escaping (ChatChannelController) -> Bool = { _ in true }
     ) {
@@ -69,6 +71,7 @@ public class Utils {
         self.messageIdBuilder = messageIdBuilder
         self.shouldSyncChannelControllerOnAppear = shouldSyncChannelControllerOnAppear
         self.sortReactions = sortReactions
+        self.channelHeaderLoader = channelHeaderLoader
         messageListDateUtils = MessageListDateUtils(messageListConfig: messageListConfig)
     }