feat: show correct error message for interval server errors

ohcnetwork · Dec 15, 2023 · b5b902f · b5b902f
1 parent 1f5ea20
commit b5b902f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 28 deletions.
diff --git a/src/Redux/fireRequest.tsx b/src/Redux/fireRequest.tsx
@@ -149,7 +149,7 @@ export const fireRequest = (
           }
 
           // 4xx Errors
-          if (error.response.status > 400 && error.response.status < 500) {
+          if (error.response.status > 400 && error.response.status < 600) {
             if (error.response.data && error.response.data.detail) {
               if (error.response.data.code === "token_not_valid") {
                 window.location.href = `/session-expired?redirect=${window.location.href}`;
@@ -167,14 +167,6 @@ export const fireRequest = (
             }
             return;
           }
-
-          // 5xx Errors
-          if (error.response.status >= 500 && error.response.status <= 599) {
-            Notification.Error({
-              msg: "Something went wrong...!",
-            });
-            return;
-          }
         } else {
           return error.response;
         }
@@ -264,7 +256,7 @@ export const fireRequestV2 = (
         }
 
         // 4xx Errors
-        if (error.response.status > 400 && error.response.status < 500) {
+        if (error.response.status > 400 && error.response.status < 600) {
           if (error.response.data && error.response.data.detail) {
             Notification.Error({
               msg: error.response.data.detail,
@@ -279,14 +271,6 @@ export const fireRequestV2 = (
           }
           return;
         }
-
-        // 5xx Errors
-        if (error.response.status >= 500 && error.response.status <= 599) {
-          Notification.Error({
-            msg: "Something went wrong...!",
-          });
-          return;
-        }
       }
     });
 };
@@ -374,7 +358,7 @@ export const fireRequestForFiles = (
           }
 
           // 4xx Errors
-          if (error.response.status > 400 && error.response.status < 500) {
+          if (error.response.status > 400 && error.response.status < 600) {
             if (error.response.status === 429) {
               return error.response;
             } else if (error.response.data && error.response.data.detail) {
@@ -388,14 +372,6 @@ export const fireRequestForFiles = (
             }
             return;
           }
-
-          // 5xx Errors
-          if (error.response.status >= 500 && error.response.status <= 599) {
-            Notification.Error({
-              msg: "Something went wrong...!",
-            });
-            return;
-          }
         }
       });
   };

diff --git a/src/Utils/request/handleResponse.ts b/src/Utils/request/handleResponse.ts
@@ -15,7 +15,7 @@ export default function handleResponse(
   // 5xx errors
   if (res.status >= 500 && res.status < 600) {
     console.error("Server error", res);
-    notify?.Error({ msg: "Something went wrong...!" });
+    notify?.Error({ msg: error?.detail || "Something went wrong...!" });
     return;
   }