Merge pull request #2731 from daostack/CW-hotfix-app-crash

[HOTFIX] App crash

Author: MeyerPV

src/pages/OldCommon/hooks/useCommonMembers.ts

@@ -1,6 +1,5 @@
 import { useCallback, useRef, useState } from "react";
 import { useDispatch, useSelector } from "react-redux";
-import { trace } from "firebase/performance";
 import { CommonService, Logger, UserService } from "@/services";
 import { store } from "@/shared/appConfig";
 import { LoadingState } from "@/shared/interfaces";
@@ -11,7 +10,6 @@ import {
   selectUserStates,
 } from "@/store/states";
 import { useDeepCompareEffect } from "react-use";
-import { perf } from "@/shared/utils/firebase";
 
 interface Options {
   commonId?: string;
@@ -113,9 +111,6 @@ export const useCommonMembers = ({ commonId }: Options): Return => {
 
     (async () => {
       try {
-        const useCommonMembersTrace = trace(perf, 'useCommonMembers');
-        useCommonMembersTrace.start();
-
         const cachedUserStates = selectUserStates()(store.getState());
         const hasUsersFromCache = commonMembers.some(
           ({ userId }) => cachedUserStates[userId]?.data,
@@ -188,7 +183,6 @@ export const useCommonMembers = ({ commonId }: Options): Return => {
           };
         });
         dispatch(cacheActions.updateUserStates(fetchedUsers));
-        useCommonMembersTrace.stop();
       } catch (err) {
         Logger.error(err);
         setState((prevState) => ({

src/shared/hooks/useCases/useDiscussionMessagesById.ts

@@ -1,7 +1,6 @@
 import { useState, useCallback } from "react";
 import { useDispatch, useSelector } from "react-redux";
 import { useDeepCompareEffect, useUpdateEffect } from "react-use";
-import { trace } from "firebase/performance";
 import {
   DiscussionMessageService,
   MESSAGES_NUMBER_IN_BATCH,
@@ -21,7 +20,7 @@ import {
   User,
 } from "@/shared/models";
 import { InternalLinkData } from "@/shared/utils";
-import firebase, { perf } from "@/shared/utils/firebase";
+import firebase from "@/shared/utils/firebase";
 import {
   cacheActions,
   selectDiscussionMessagesStateByDiscussionId,
@@ -154,10 +153,7 @@ export const useDiscussionMessagesById = ({
 
   const fetchRepliedMessages = useCallback(
     async (messageId: string, endDate: Date): Promise<void> => {
-      const fetchRepliedMessagesTrace = trace(perf, 'fetchRepliedMessagesTrace');
       try {
-        fetchRepliedMessagesTrace.start();
-
         if (state.data?.find((item) => item.id === messageId)) {
           return Promise.resolve();
         }
@@ -211,9 +207,8 @@ export const useDiscussionMessagesById = ({
             updatedDiscussionMessages: discussionsWithText,
           }),
         );
-        fetchRepliedMessagesTrace.stop();
       } catch(err) {
-        fetchRepliedMessagesTrace.stop();
+        // console.log(err);
       }
     },
     [
@@ -248,9 +243,6 @@ export const useDiscussionMessagesById = ({
     }
 
     try {
-      const fetchDiscussionMessagesTrace = trace(perf, 'fetchDiscussionMessages');
-      fetchDiscussionMessagesTrace.start();
-
       DiscussionMessageService.subscribeToDiscussionMessagesByDiscussionId(
         discussionId,
         lastVisible && lastVisible[discussionId],
@@ -320,7 +312,6 @@ export const useDiscussionMessagesById = ({
           setIsBatchLoading(false);
         },
       );
-      fetchDiscussionMessagesTrace.stop();
     } catch(err) {
       setIsBatchLoading(false);
     }

src/shared/layouts/MultipleSpacesLayout/components/Header/components/Breadcrumbs/components/FeedItemBreadcrumbs/FeedItemBreadcrumbs.tsx

@@ -67,7 +67,7 @@ const FeedItemBreadcrumbs: FC<FeedItemBreadcrumbsProps> = ({
     });
 
     return () => {
-      unsubscribe();
+      unsubscribe && unsubscribe();
     };
   }, [breadcrumbs.activeItem?.id]);