Merge pull request #168 from frappe/handle-network-errors

frappe · Sep 25, 2024 · c34ce58 · c34ce58
2 parents a514718 + c4f145f
commit c34ce58
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 13 deletions.
diff --git a/src/utils/frappeRequest.js b/src/utils/frappeRequest.js
@@ -94,5 +94,9 @@ export function frappeRequest(options) {
         throw e
       }
     },
+    transformError: (error) => {
+      options.onError && options.onError(error)
+      throw error
+    },
   })
 }
diff --git a/src/utils/request.js b/src/utils/request.js
@@ -31,19 +31,26 @@ export function request(_options) {
     method: options.method || 'GET',
     headers: options.headers,
     body,
-  }).then((response) => {
-    if (options.transformResponse) {
-      return options.transformResponse(response, options)
-    }
-    if (response.status >= 200 && response.status < 300) {
-      if (options.responseType === 'json') {
-        return response.json()
+  })
+    .then((response) => {
+      if (options.transformResponse) {
+        return options.transformResponse(response, options)
+      }
+      if (response.status >= 200 && response.status < 300) {
+        if (options.responseType === 'json') {
+          return response.json()
+        }
+        return response
+      } else {
+        let error = new Error(response.statusText)
+        error.response = response
+        throw error
+      }
+    })
+    .catch((error) => {
+      if (options.transformError) {
+        return options.transformError(error)
       }
-      return response
-    } else {
-      let error = new Error(response.statusText)
-      error.response = response
       throw error
-    }
-  })
+    })
 }