Only start the task to get updated chats after all chats have been re…

…trieved (#77)
open-chat-labs · Jan 15, 2021 · 01f2256 · 01f2256
1 parent e1d00e8
commit 01f2256
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/website/public/backgroundTasks.ts b/src/website/public/backgroundTasks.ts
@@ -79,10 +79,10 @@ export function setupBackgroundTasks() {
 
     // Check for new messages at regular intervals
     useEffect(() => {
-        if (usersState.me) {
+        if (chatsState.runUpdateChatsTask) {
             return updateChatsRegularlyTask(chatsState.chatsSyncedUpTo, dispatch);
         }
-    }, [usersState.me]);
+    }, [chatsState.runUpdateChatsTask]);
 }
 
 function updateChatsRegularlyTask(chatsSyncedUpTo: Option<Timestamp>, dispatch: Dispatch<any>) : () => void {

diff --git a/src/website/public/reducers/chatsReducer.ts b/src/website/public/reducers/chatsReducer.ts
@@ -62,13 +62,15 @@ import {
 export type ChatsState = {
     chats: Chat[],
     selectedChatIndex: Option<number>,
-    chatsSyncedUpTo: Option<Timestamp>
+    chatsSyncedUpTo: Option<Timestamp>,
+    runUpdateChatsTask: boolean
 }
 
 const initialState: ChatsState = {
     chats: [],
     selectedChatIndex: null,
-    chatsSyncedUpTo: null
+    chatsSyncedUpTo: null,
+    runUpdateChatsTask: false
 };
 
 type Event =
@@ -141,7 +143,8 @@ export default produce((state: ChatsState, event: Event) => {
             return {
                 chats,
                 selectedChatIndex: chats.length ? 0 : null,
-                chatsSyncedUpTo: latestUpdateTimestamp
+                chatsSyncedUpTo: latestUpdateTimestamp,
+                runUpdateChatsTask: true
             };
         }