fix: Add optional chaining for undefined logger and pubSub (#1270)

Author: HoonBaek

src/modules/OpenChannel/context/hooks/useInitialMessagesFetch.ts

@@ -12,7 +12,7 @@ interface DynamicParams {
   userFilledMessageListParams?: Record<string, any>;
 }
 interface StaticParams {
-  logger: Logger;
+  logger?: Logger;
   messagesDispatcher: CustomUseReducerDispatcher;
   scrollRef: React.RefObject<HTMLElement>;
 }
@@ -22,7 +22,7 @@ function useInitialMessagesFetch(
   { logger, messagesDispatcher, scrollRef }: StaticParams,
 ): void {
   useEffect(() => {
-    logger.info('OpenChannel | useInitialMessagesFetch: Setup started', currentOpenChannel);
+    logger?.info('OpenChannel | useInitialMessagesFetch: Setup started', currentOpenChannel);
     messagesDispatcher({
       type: messageActionTypes.RESET_MESSAGES,
       payload: null,
@@ -40,16 +40,16 @@ function useInitialMessagesFetch(
           // @ts-ignore
           messageListParams[key] = userFilledMessageListParams[key];
         });
-        logger.info('OpenChannel | useInitialMessagesFetch: Used customizedMessageListParams');
+        logger?.info('OpenChannel | useInitialMessagesFetch: Used customizedMessageListParams');
       }
 
-      logger.info('OpenChannel | useInitialMessagesFetch: Fetching messages', { currentOpenChannel, messageListParams });
+      logger?.info('OpenChannel | useInitialMessagesFetch: Fetching messages', { currentOpenChannel, messageListParams });
       messagesDispatcher({
         type: messageActionTypes.GET_PREV_MESSAGES_START,
         payload: null,
       });
       currentOpenChannel.getMessagesByTimestamp(new Date().getTime(), messageListParams).then((messages) => {
-        logger.info('OpenChannel | useInitialMessagesFetch: Fetching messages succeeded', messages);
+        logger?.info('OpenChannel | useInitialMessagesFetch: Fetching messages succeeded', messages);
         const hasMore = (messages && messages.length > 0);
         const lastMessageTimestamp = hasMore ? messages[0].createdAt : null;
         messagesDispatcher({
@@ -63,7 +63,7 @@ function useInitialMessagesFetch(
         });
         setTimeout(() => { scrollIntoLast(0, scrollRef); });
       }).catch((error) => {
-        logger.error('OpenChannel | useInitialMessagesFetch: Fetching messages failed', error);
+        logger?.error('OpenChannel | useInitialMessagesFetch: Fetching messages failed', error);
         messagesDispatcher({
           type: messageActionTypes.GET_PREV_MESSAGES_FAIL,
           payload: {

src/modules/OpenChannelApp/index.tsx

@@ -31,7 +31,7 @@ export default function OpenChannelApp({
   const [showSettings, setShowSettings] = useState(false);
 
   return (
-    <Sendbird appId={appId} userId={userId} nickname={nickname} theme={theme} imageCompression={imageCompression}>
+    <Sendbird appId={appId} userId={userId} nickname={nickname} theme={theme} imageCompression={imageCompression} config={{ logLevel: 'all' }}>
       <div className="sendbird-openchannel-app">
         <div className={'sendbird-openchannel-app__channellist'}>
           <OpenChannelList onChannelSelected={(channel) => setChannelUrl(channel.url)} />

src/modules/OpenChannelList/context/OpenChannelListProvider.tsx

@@ -61,19 +61,21 @@ export const OpenChannelListProvider: React.FC<OpenChannelListProviderProps> = (
   // Events & PubSub
   useEffect(() => {
     const subscriber = new Map<string, { remove:() => void }>();
-    subscriber.set(
-      pubSubTopics.UPDATE_OPEN_CHANNEL,
-      pubSub.subscribe(pubSubTopics.UPDATE_OPEN_CHANNEL, (channel) => {
-        openChannelListDispatcher({
-          type: OpenChannelListActionTypes.UPDATE_OPEN_CHANNEL,
-          payload: channel,
-        });
-      }),
-    );
+    if (pubSub?.subscribe) {
+      subscriber.set(
+        pubSubTopics.UPDATE_OPEN_CHANNEL,
+        pubSub.subscribe(pubSubTopics.UPDATE_OPEN_CHANNEL, (channel) => {
+          openChannelListDispatcher({
+            type: OpenChannelListActionTypes.UPDATE_OPEN_CHANNEL,
+            payload: channel,
+          });
+        }),
+      );
+    }
     return () => {
-      subscriber.forEach((it) => it.remove());
+      subscriber.forEach((it) => it?.remove());
     };
-  }, [sdkInitialized, pubSub]);
+  }, [sdkInitialized, pubSub?.subscribe]);
 
   // Fetch next channels by scroll event
   const fetchNextChannels = useFetchNextCallback({

src/modules/OpenChannelList/context/hooks/useSetupOpenChannelList.ts

@@ -12,7 +12,7 @@ interface DynamicParams {
   openChannelListQuery?: UserFilledOpenChannelListQuery;
 }
 interface StaticParams {
-  logger: Logger;
+  logger?: Logger;
   openChannelListDispatcher: OpenChannelListDispatcherType;
 }
 
@@ -30,7 +30,7 @@ function useSetupOpenChannelList(
   useEffect(() => {
     if (sdkInitialized) {
       if (sdk?.openChannel) {
-        logger.info('OpenChannelList|useSetupOpenChannelList: Setup OpenChannelList', { sdkInitialized });
+        logger?.info('OpenChannelList|useSetupOpenChannelList: Setup OpenChannelList', { sdkInitialized });
         const channelListQuery = createChannelListQuery({
           sdk,
           logger,
@@ -40,34 +40,34 @@ function useSetupOpenChannelList(
         });
 
         if (channelListQuery?.hasNext) {
-          logger.info('OpenChannelList|useSetupOpenChannelList: Fetch channels');
+          logger?.info('OpenChannelList|useSetupOpenChannelList: Fetch channels');
           openChannelListDispatcher({
             type: OpenChannelListActionTypes.INIT_OPEN_CHANNEL_LIST_START,
             payload: null,
           });
           channelListQuery.next()
             .then((channelList) => {
-              logger.info('OpenChannelList|useSetupOpenChannelList: Succeeded fetching channels', channelList);
+              logger?.info('OpenChannelList|useSetupOpenChannelList: Succeeded fetching channels', channelList);
               openChannelListDispatcher({
                 type: OpenChannelListActionTypes.INIT_OPEN_CHANNEL_LIST_SUCCESS,
                 payload: channelList,
               });
             })
             .catch((err) => {
-              logger.error('OpenChannelList|useSetupOpenChannelList: Failed fetching channels', err);
+              logger?.error('OpenChannelList|useSetupOpenChannelList: Failed fetching channels', err);
               openChannelListDispatcher({
                 type: OpenChannelListActionTypes.INIT_OPEN_CHANNEL_LIST_FAILURE,
                 payload: null,
               });
             });
         } else {
-          logger.info('OpenChannelList|useSetupOpenChannelList: There is no more channels');
+          logger?.info('OpenChannelList|useSetupOpenChannelList: There is no more channels');
         }
       } else {
-        logger.warning('OpenChannelList|useSetupOpenChannelList: openChannel is not included in the Chat SDK', sdk);
+        logger?.warning('OpenChannelList|useSetupOpenChannelList: openChannel is not included in the Chat SDK', sdk);
       }
     } else {
-      logger.info('OpenChannelList|useSetupOpenChannelList: Reset OpenChannelList', { sdkInitialized });
+      logger?.info('OpenChannelList|useSetupOpenChannelList: Reset OpenChannelList', { sdkInitialized });
       openChannelListDispatcher({
         type: OpenChannelListActionTypes.RESET_OPEN_CHANNEL_LIST,
         payload: null,