fix bug where errors would not show until restart

jmicko · Jan 23, 2024 · 89a1e6b · 89a1e6b
1 parent f51ca7f
commit 89a1e6b
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 9 deletions.
diff --git a/client/src/components/Messages/MessageSection.tsx b/client/src/components/Messages/MessageSection.tsx
@@ -62,6 +62,8 @@ export function MessageSection({
     }
   }, [messages, initialRender, mobileCollapsed, header]);
 
+  console.log(messages, 'messages ' + header);
+
 
   return (
     (user.can_chat || header !== 'Chat') &&
@@ -70,12 +72,13 @@ export function MessageSection({
       : 'expanded'} ` + header.split(' ').join('-').toLowerCase()}>
 
 
-      <h3 className={
-        `${(collapsed && !onMobile)
-          || (mobileCollapsed !== header && mobileCollapsed !== '' && onMobile)
-          ? 'collapsed'
-          : 'expanded'} title message-header ${user.theme}`
-      }
+      <h3
+        className={
+          `${(collapsed && !onMobile)
+            || (mobileCollapsed !== header && mobileCollapsed !== '' && onMobile)
+            ? 'collapsed'
+            : 'expanded'} title message-header ${user.theme}`
+        }
         onClick={() => {
           if (onMobile) {
             if (mobileCollapsed === header) {
@@ -84,7 +87,7 @@ export function MessageSection({
               console.log('setting mobile collapsed to ', header);
               setMobileCollapsed && setMobileCollapsed(header)
             }
-          }
+          } header
         }}
       >
         {(collapsed && !onMobile) && messages.length} {header}
@@ -100,6 +103,7 @@ export function MessageSection({
           })
         }
         <div className="spacer" />
+        {/* {JSON.stringify(messages)} */}
       </div>}
 
       {header === 'Chat' &&

diff --git a/client/src/providers/DataProvider.tsx b/client/src/providers/DataProvider.tsx
@@ -114,6 +114,9 @@ export function DataProvider({ children }: { children: ReactNode }) {
     refresh: refreshBotErrors,
   } = useGetFetch<Messages>(botErrorsOptions)
 
+  console.log(botErrors, '< botErrors');
+
+
   // ////////////////////////
   // //////// ORDERS ////////
   // ////////////////////////

diff --git a/server/modules/cache.js b/server/modules/cache.js
@@ -351,7 +351,7 @@ class Messenger {
     this.instantMessage({ type: 'messageUpdate', messageUpdate: true })
   }
   // todo - should probably use type: 'error' and get rid of this
-  newError(err) {
+  async newError(err) {
     try {
       devLog(err.errorText);
       const error = new Message({
@@ -360,7 +360,7 @@ class Messenger {
         data: err.data ? err.data : null
       });
       if (error.text) {
-        const saved = saveMessage(this.userID, error);
+        const saved = await saveMessage(this.userID, error);
         this.errors.unshift(saved);
       }
 
@@ -379,6 +379,15 @@ class Messenger {
   }
   getErrors() {
     return this.errors;
+    // const errors = [];
+
+    // this.messages.forEach(message => {
+    //   // console.log(message, 'message');
+    //   if (message.type === 'error') {
+    //     errors.push(message);
+    //   }
+    // });
+    // return errors;
   }
   clearErrors() {
     this.errors.length = 0;

diff --git a/server/routes/account.router.js b/server/routes/account.router.js
@@ -450,6 +450,7 @@ router.get('/errors', rejectUnauthenticated, async (req, res) => {
   const userID = req.user.id;
   try {
     const userErrors = messenger[userID].getErrors();
+    console.log(userErrors, '< userErrors');
     res.send(userErrors);
   } catch (err) {
     devLog(err, 'problem debug route');