bugfix: Migrate new AppState subs

hyochan · Jun 7, 2022 · 53f498b · 53f498b
1 parent 078aa84
commit 53f498b
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 11 deletions.
diff --git a/client/package.json b/client/package.json
@@ -1,7 +1,7 @@
 {
   "name": "hackatalk",
   "private": true,
-  "version": "2.1.0",
+  "version": "2.1.1",
   "main": "node_modules/expo/AppEntry.js",
   "description": "Opensource chat app",
   "author": "dooboolab",

diff --git a/client/src/components/pages/Message.tsx b/client/src/components/pages/Message.tsx
@@ -41,7 +41,6 @@ import React, {
   FC,
   ReactElement,
   Suspense,
-  memo,
   useEffect,
   useLayoutEffect,
   useMemo,
@@ -601,7 +600,8 @@ const ContentContainer: FC<ContentProps> = ({searchArgs, channelId}) => {
 
   useLayoutEffect(() => {
     navigation.setOptions({
-      headerTitle: memo((): ReactElement => {
+      // eslint-disable-next-line react/no-unstable-nested-components
+      headerTitle: (): ReactElement => {
         let title = channel?.name || '';
 
         // Note that if the user exists, this is direct message which title should appear as user name or nickname
@@ -633,7 +633,7 @@ const ContentContainer: FC<ContentProps> = ({searchArgs, channelId}) => {
             {title}
           </Text>
         );
-      }),
+      },
     });
   }, [auth?.id, channel?.name, navigation, users]);
 
@@ -694,7 +694,9 @@ const MessageScreen: FC = () => {
       });
     }
 
-    deleteSameChannelNotification();
+    if (Platform.OS !== 'web') {
+      deleteSameChannelNotification();
+    }
   }, [channelId]);
 
   const searchArgs: MessagesQuery$variables = {

diff --git a/client/src/hooks/useAppStateChangeHandler.tsx b/client/src/hooks/useAppStateChangeHandler.tsx
@@ -12,10 +12,10 @@ export default function useAppStateChangeHandler(
   handler: AppStateChangeHandler,
 ): void {
   useEffect(() => {
-    AppState.addEventListener('change', handler);
+    const subscription = AppState.addEventListener('change', handler);
 
     return (): void => {
-      AppState.removeEventListener('change', handler);
+      subscription.remove();
     };
   }, [handler]);
 }
diff --git a/server/src/utils/jwt.ts b/server/src/utils/jwt.ts
@@ -88,6 +88,8 @@ export const getRefreshToken = async (
   return !result.verified ? null : refreshToken;
 };
 
+const {NODE_ENV} = process.env;
+
 export const refresh = async (
   userId: string,
   prisma: PrismaClient,
@@ -96,10 +98,17 @@ export const refresh = async (
     expiresIn: REFRESH_TIME,
   });
 
-  await prisma.profile.update({
-    data: {refreshToken: newToken},
-    where: {userId},
-  });
+  try {
+    await prisma.profile.update({
+      data: {refreshToken: newToken},
+      where: {userId},
+    });
+  } catch (err) {
+    if (NODE_ENV !== 'production') {
+      // eslint-disable-next-line no-console
+      console.log('profile update err', err);
+    }
+  }
 
   return newToken;
 };