Fixed loading of pinned messages in the channel info screen (#462)

Author: martinmitrevski

CHANGELOG.md

@@ -8,6 +8,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 ### 🐞 Fixed
 - Reactions picker for large messages sometimes goes in the safe area
+- Loading of pinned messages in the channel info screen
 
 # [4.50.1](https://github.com/GetStream/stream-chat-swiftui/releases/tag/4.50.1)
 _March 14, 2024_

Sources/StreamChatSwiftUI/ChatChannel/ChannelInfo/ChatChannelInfoHelperViews.swift

@@ -88,7 +88,10 @@ public struct ChatInfoOptionsView: View {
                 icon: images.pin,
                 title: L10n.ChatInfo.PinnedMessages.title
             ) {
-                PinnedMessagesView(channel: viewModel.channel)
+                PinnedMessagesView(
+                    channel: viewModel.channel,
+                    channelController: viewModel.channelController
+                )
             }
 
             Divider()

Sources/StreamChatSwiftUI/ChatChannel/ChannelInfo/ChatChannelInfoViewModel.swift

@@ -45,7 +45,7 @@ public class ChatChannelInfoViewModel: ObservableObject, ChatChannelControllerDe
         channel.ownCapabilities.contains(.updateChannel)
     }
 
-    private var channelController: ChatChannelController!
+    var channelController: ChatChannelController!
     private var memberListController: ChatChannelMemberListController!
     private var loadingUsers = false
 

Sources/StreamChatSwiftUI/ChatChannel/ChannelInfo/PinnedMessagesView.swift

@@ -10,9 +10,12 @@ public struct PinnedMessagesView: View {
 
     @StateObject private var viewModel: PinnedMessagesViewModel
 
-    public init(channel: ChatChannel) {
+    public init(channel: ChatChannel, channelController: ChatChannelController? = nil) {
         _viewModel = StateObject(
-            wrappedValue: PinnedMessagesViewModel(channel: channel)
+            wrappedValue: PinnedMessagesViewModel(
+                channel: channel,
+                channelController: channelController
+            )
         )
     }
 

Sources/StreamChatSwiftUI/ChatChannel/ChannelInfo/PinnedMessagesViewModel.swift

@@ -12,9 +12,34 @@ public class PinnedMessagesViewModel: ObservableObject {
     private let channel: ChatChannel
 
     @Published var pinnedMessages: [ChatMessage]
-
-    public init(channel: ChatChannel) {
+    
+    private var channelController: ChatChannelController?
+    
+    public init(channel: ChatChannel, channelController: ChatChannelController? = nil) {
         self.channel = channel
-        pinnedMessages = channel.pinnedMessages
+        if channelController != nil {
+            pinnedMessages = []
+        } else {
+            pinnedMessages = channel.pinnedMessages
+        }
+        self.channelController = channelController
+        loadPinnedMessages()
+    }
+    
+    // MARK: - private
+    
+    private func loadPinnedMessages() {
+        channelController?.loadPinnedMessages(completion: { [weak self] result in
+            switch result {
+            case let .success(messages):
+                withAnimation {
+                    self?.pinnedMessages = messages
+                }
+                log.debug("Successfully loaded pinned messages")
+            case let .failure(error):
+                self?.pinnedMessages = self?.channel.pinnedMessages ?? []
+                log.error("Error loading pinned messages \(error.localizedDescription)")
+            }
+        })
     }
 }

Sources/StreamChatSwiftUI/ChatChannel/Reactions/ReactionsOverlayView.swift

@@ -281,7 +281,7 @@ public struct ReactionsOverlayView<Factory: ViewFactory>: View {
             messageDisplayInfo.showsMessageActions ? messageActionsSize : userReactionsPopupHeight
         var originY = messageDisplayInfo.frame.origin.y
         let minOrigin: CGFloat = 100
-        let maxOrigin: CGFloat = screenHeight + topSafeArea - messageContainerHeight - bottomPopupOffset - minOrigin - bottomOffset
+        let maxOrigin: CGFloat = screenHeight - messageContainerHeight - bottomPopupOffset - minOrigin - bottomOffset
         if originY < minOrigin {
             originY = minOrigin
         } else if originY > maxOrigin {