Handle update errors in the UI.

Oxygem · Jul 31, 2021 · fec24c8 · fec24c8
1 parent f882d40
commit fec24c8
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 0 deletions.
diff --git a/kanmail/client/components/emails/Filters.jsx b/kanmail/client/components/emails/Filters.jsx
@@ -128,6 +128,7 @@ export default class Filters extends React.Component {
         folders: PropTypes.array.isRequired,
         accountName: PropTypes.string,
         updateVersion: PropTypes.string,
+        updateError: PropTypes.string,
     }
 
     constructor(props) {
@@ -241,6 +242,16 @@ export default class Filters extends React.Component {
     }
 
     renderUpdateLink() {
+        if (this.props.updateError) {
+            return (
+                <li><a onClick={() => openLink('https://kanmail.io/download')}>
+                    <span className="update error">
+                        Error downloading update: <code>{this.props.updateError}</code>.<br />Please re-download the latest version from the Kanmail website.
+                    </span>
+                </a></li>
+            );
+        }
+
         if (!this.props.updateVersion) {
             return;
         }

diff --git a/kanmail/client/stores/update.js b/kanmail/client/stores/update.js
@@ -17,6 +17,7 @@ class UpdateStore extends BaseStore {
             updateVersion: null,
             updateReady: false,
             updateDownloading: false,
+            updateError: null,
         };
     }
 
@@ -37,6 +38,10 @@ class UpdateStore extends BaseStore {
         return post('/api/update').then(() => {
             this.props.updateReady = true;
             this.triggerUpdate();
+        }).catch((e) => {
+            this.props.updateDownloading = false;
+            this.props.updateError = e.data ? e.data.errorMessage : 'unknown';
+            this.triggerUpdate();
         });
     }
 }

diff --git a/kanmail/client/styles/themes/base.less b/kanmail/client/styles/themes/base.less
@@ -205,6 +205,10 @@ section#sidebar {
             span.update {
                 background: @green;
                 color: white;
+
+                &.error {
+                    background: @red;
+                }
             }
 
             a {