From e3ab5afcbdc257bba80d759678838e87fb5f8e10 Mon Sep 17 00:00:00 2001 From: Mouse Reeve Date: Fri, 10 Sep 2021 08:59:49 -0700 Subject: [PATCH] Catch error responses from API --- bookwyrm/static/js/status_cache.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/bookwyrm/static/js/status_cache.js b/bookwyrm/static/js/status_cache.js index 19fa8d5c..2da509c4 100644 --- a/bookwyrm/static/js/status_cache.js +++ b/bookwyrm/static/js/status_cache.js @@ -74,11 +74,16 @@ let StatusCache = new class { BookWyrm.addRemoveClass(form, 'is-processing', false); trigger.removeAttribute('disabled'); }) - .then(this.submitStatusSuccess.bind(this, form)) + .then(response => { + if (!response.ok) { + throw new Error(); + } + this.submitStatusSuccess(form); + }) .catch(error => { // @todo Display a notification in the UI instead. // For now, the absence of change will be enough. - console.warn('Request failed:', error); + console.log('Request failed:', error); BookWyrm.addRemoveClass(form, 'has-error', form.className.indexOf('is-hidden') == -1); });