stream-chat-react-native-core 5.3.0-beta.1 → 5.3.0

package/src/components/ChannelList/ChannelListMessenger.tsx

@@ -132,14 +132,14 @@ const ChannelListMessengerWithContext = <
     if (debugRef.current.setSendEventParams)
       debugRef.current.setSendEventParams({
         action: 'Channels',
-        data: channels.map((channel) => ({
+        data: channels?.map((channel) => ({
           data: channel.data,
           members: channel.state.members,
         })),
       });
   }
 
-  if (error && !refreshing && !loadingChannels && !channels?.length) {
+  if (error && !refreshing && !loadingChannels && channels === null) {
     return (
       <LoadingErrorIndicator
         error={error}
@@ -157,7 +157,7 @@ const ChannelListMessengerWithContext = <
   };
 
   const ListFooterComponent = () =>
-    channels.length && ListHeaderComponent ? <ListHeaderComponent /> : null;
+    channels?.length && ListHeaderComponent ? <ListHeaderComponent /> : null;
 
   if (loadingChannels) {
     return <LoadingIndicator listType='channel' />;

package/src/components/ChannelList/__tests__/ChannelList.test.js

@@ -43,7 +43,7 @@ const ChannelListComponent = (props) => {
   const { channels, onSelect } = useChannelsContext();
   return (
     <View testID='channel-list'>
-      {channels.map((channel) => (
+      {channels?.map((channel) => (
         <ChannelPreviewComponent
           {...props}
           channel={channel}

package/src/components/ChannelList/hooks/useCreateChannelsContext.ts

@@ -38,7 +38,7 @@ export const useCreateChannelsContext = <
   Skeleton,
 }: ChannelsContextValue<StreamChatGenerics>) => {
   const channelValueString = channels
-    .map(
+    ?.map(
       (channel) =>
         `${channel.data?.name ?? ''}${channel.id ?? ''}${Object.values(channel.state.members)
           .map((member) => member.user?.online)

package/src/components/ChannelList/hooks/usePaginatedChannels.ts

@@ -44,11 +44,8 @@ export const usePaginatedChannels = <
   sort = {},
 }: Parameters<StreamChatGenerics>) => {
   const { client } = useChatContext<StreamChatGenerics>();
-  const initialChannelsStateRef = useRef([]);
-  const [channels, setChannels] = useState<Channel<StreamChatGenerics>[]>(
-    initialChannelsStateRef.current,
-  );
-  const [staticChannelsActive, setStaticChannelsActive] = useState<boolean>(true);
+  const [channels, setChannels] = useState<Channel<StreamChatGenerics>[] | null>(null);
+  const [staticChannelsActive, setStaticChannelsActive] = useState<boolean>(false);
   const activeChannels = useActiveChannelsRefContext();
 
   const [error, setError] = useState<Error>();
@@ -95,7 +92,8 @@ export const usePaginatedChannels = <
 
     const newOptions = {
       limit: options?.limit ?? MAX_QUERY_CHANNELS_LIMIT,
-      offset: queryType === 'loadChannels' && !staticChannelsActive ? channels.length : 0,
+      offset:
+        queryType === 'loadChannels' && !staticChannelsActive && channels ? channels.length : 0,
       ...options,
     };
 
@@ -108,7 +106,7 @@ export const usePaginatedChannels = <
         return;
       }
       const newChannels =
-        queryType === 'loadChannels' && !staticChannelsActive
+        queryType === 'loadChannels' && !staticChannelsActive && channels
           ? [...channels, ...channelQueryResponse]
           : channelQueryResponse;
 
@@ -195,6 +193,7 @@ export const usePaginatedChannels = <
                 offlineMode: true,
               }),
             );
+            setStaticChannelsActive(true);
           }
         } catch (e) {
           console.warn('Failed to get channels from database: ', e);
@@ -212,15 +211,17 @@ export const usePaginatedChannels = <
     error,
     hasNextPage,
     loadingChannels:
-      activeQueryType === 'queryLocalDB'
-        ? true
-        : activeQueryType === 'reload' && channels === initialChannelsStateRef.current,
+      activeQueryType === 'queryLocalDB' ? true : activeQueryType === 'reload' && channels === null,
     loadingNextPage: activeQueryType === 'loadChannels',
     loadNextPage,
     refreshing: activeQueryType === 'refresh',
     refreshList,
     reloadList,
-    setChannels,
+    // Although channels can be null, there is no practical case where channels will be null
+    // when setChannels is used. setChannels is only recommended to be used for overriding
+    // event handler. Thus instead of adding if check for channels === null, its better to
+    // simply reassign types here.
+    setChannels: setChannels as React.Dispatch<React.SetStateAction<Channel<StreamChatGenerics>[]>>,
     staticChannelsActive,
   };
 };

package/src/contexts/channelsContext/ChannelsContext.tsx

@@ -48,7 +48,7 @@ export type ChannelsContextValue<
   /**
    * Channels can be either an array of channels or a promise which resolves to an array of channels
    */
-  channels: Channel<StreamChatGenerics>[];
+  channels: Channel<StreamChatGenerics>[] | null;
   /**
    * Custom indicator to use when channel list is empty
    *

package/src/native.ts

@@ -27,7 +27,7 @@ export let compressImage: CompressImage = fail;
 type DeleteFile = ({ uri }: { uri: string }) => Promise<boolean> | never;
 export let deleteFile: DeleteFile = fail;
 
-type GetLocalAssetUri = (uriOrAssetId: string) => never;
+type GetLocalAssetUri = (uriOrAssetId: string) => Promise<string> | never;
 export let getLocalAssetUri: GetLocalAssetUri = fail;
 
 type GetPhotos = ({ after, first }: { first: number; after?: string }) =>
@@ -157,7 +157,7 @@ export type SoundType = {
     source?: { uri: string },
     initialStatus?: Partial<AVPlaybackStatusToSet>,
     onPlaybackStatusUpdate?: (playbackStatus: PlaybackStatus) => void,
-  ) => SoundReturnType | null;
+  ) => Promise<SoundReturnType | null>;
   Player: React.ComponentType<SoundReturnType> | null;
 };
 

package/src/version.json

@@ -1,3 +1,3 @@
 {
-  "version": "5.3.0-beta.1"
+  "version": "5.3.0"
 }