Reject responses that aren't 200s

nyaruka · Nov 8, 2024 · 2848e1c · 2848e1c
1 parent 89f6d46
commit 2848e1c
Show file tree

Hide file tree

Showing 6 changed files with 32 additions and 15 deletions.
diff --git a/src/dialog/Modax.ts b/src/dialog/Modax.ts
@@ -278,8 +278,8 @@ export class Modax extends RapidElement {
     // this.cancelToken = CancelToken.source();
     this.fetching = true;
     this.body = this.getLoading();
-    getUrl(this.endpoint, null, this.getHeaders()).then(
-      (response: WebResponse) => {
+    getUrl(this.endpoint, null, this.getHeaders())
+      .then((response: WebResponse) => {
         // if it's a full page, breakout of the modal
         if (response.body.indexOf('<!DOCTYPE HTML>') == 0) {
           this.open = false;
@@ -294,8 +294,12 @@ export class Modax extends RapidElement {
             });
           });
         }
-      }
-    );
+      })
+      .catch((error) => {
+        this.fetching = false;
+        this.open = false;
+        this.fireCustomEvent(CustomEventType.Error, { error });
+      });
   }
 
   public submit(extra = {}): void {

diff --git a/src/interfaces.ts b/src/interfaces.ts
@@ -279,5 +279,6 @@ export enum CustomEventType {
   DragStart = 'temba-drag-start',
   DragStop = 'temba-drag-stop',
   Resized = 'temba-resized',
-  DetailsChanged = 'temba-details-changed'
+  DetailsChanged = 'temba-details-changed',
+  Error = 'temba-error'
 }
diff --git a/src/list/ContentMenu.ts b/src/list/ContentMenu.ts
@@ -129,6 +129,11 @@ export class ContentMenu extends RapidElement {
           });
         })
         .catch((error: any) => {
+          this.fireCustomEvent(CustomEventType.Loaded, {
+            buttons: this.buttons,
+            items: this.items
+          });
+
           console.error(error);
         });
     }

diff --git a/src/list/TembaList.ts b/src/list/TembaList.ts
@@ -308,7 +308,7 @@ export class TembaList extends RapidElement {
         this.items = newItems;
       }
     } catch (error) {
-      // console.error(error);
+      this.paused = true;
     }
   }
 
@@ -332,7 +332,6 @@ export class TembaList extends RapidElement {
 
       try {
         const page = await fetchResultsPage(endpoint, controller);
-
         // sanitize our options if necessary
         if (this.sanitizeOption) {
           page.results.forEach(this.sanitizeOption);
@@ -350,10 +349,10 @@ export class TembaList extends RapidElement {
       } catch (error) {
         // aborted
         this.reset();
-        console.log(error);
+        this.paused = true;
+        this.loading = false;
         return;
       }
-
       this.nextPage = nextPage;
     }
     this.pages = pages;

diff --git a/src/list/TembaMenu.ts b/src/list/TembaMenu.ts
@@ -772,8 +772,8 @@ export class TembaMenu extends ResizeElement {
   private loadItems(item: MenuItem, event: MouseEvent = null) {
     if (item && item.endpoint) {
       item.loading = true;
-      this.httpComplete = fetchResults(item.endpoint).then(
-        (items: MenuItem[]) => {
+      this.httpComplete = fetchResults(item.endpoint)
+        .then((items: MenuItem[]) => {
           items.forEach((newItem) => {
             if (!newItem.items) {
               const prevItem = (item.items || []).find(
@@ -811,8 +811,10 @@ export class TembaMenu extends ResizeElement {
 
           this.requestUpdate('root');
           this.scrollSelectedIntoView();
-        }
-      );
+        })
+        .catch((error) => {
+          this.fireCustomEvent(CustomEventType.Error, { error });
+        });
     }
   }
 

diff --git a/src/utils/index.ts b/src/utils/index.ts
@@ -123,6 +123,11 @@ export const getUrl = (
 
     fetch(url, options)
       .then((response) => {
+        if (response.status < 200 || response.status >= 300) {
+          reject(response);
+          return;
+        }
+
         response.text().then((body: string) => {
           let json = {};
           try {
@@ -177,7 +182,9 @@ export const fetchResultsPage = (
           next: response.json.next
         });
       })
-      .catch((error) => reject(error));
+      .catch((error) => {
+        return reject(error);
+      });
   });
 };
 
@@ -326,7 +333,6 @@ export const postFormData = (
         }
       })
       .catch((err) => {
-        console.error(err);
         reject(err);
       });
   });