feat(ux): better error messages

assets/locales/en.json

@@ -141,5 +141,13 @@
   "runGarbageCollectorDone": {
     "title": "Garbage collector",
     "message": "The garbage collector ran successfully."
+  },
+  "moveRepositoryFailed": {
+    "title": "Move repository",
+    "message": "Could not move the repository from { currDir } to { newDir }."
+  },
+  "cantAddIpfsToPath": {
+    "title": "IPFS on PATH",
+    "message": "Could not add IPFS to the PATH."
   }
 }

src/dialogs/errors.js

@@ -44,8 +44,11 @@ function criticalErrorDialog (e) {
   app.exit(1)
 }
 
-function recoverableErrorDialog (e) {
-  const option = dialog({
+// Shows a recoverable error dialog with the default title and message.
+// Passing an options object alongside the error can be used to override
+// the title and message.
+function recoverableErrorDialog (e, options = {}) {
+  const cfg = {
     title: i18n.t('recoverableErrorDialog.title'),
     message: i18n.t('recoverableErrorDialog.message'),
     type: 'error',
@@ -54,7 +57,17 @@ function recoverableErrorDialog (e) {
       i18n.t('reportTheError'),
       i18n.t('openLogs')
     ]
-  })
+  }
+
+  if (options.title) {
+    cfg.title = options.title
+  }
+
+  if (options.message) {
+    cfg.message = options.message
+  }
+
+  const option = dialog(cfg)
 
   if (option === 1) {
     shell.openExternal(`https://github.com/ipfs-shipyard/ipfs-desktop/issues/new?body=${encodeURI(issueTemplate(e))}`)

src/ipfs-on-path/index.js

@@ -1,4 +1,5 @@
 const { join } = require('path')
+const i18n = require('i18next')
 const which = require('which')
 const { execFile } = require('child_process')
 const createToggler = require('../create-toggler')
@@ -53,7 +54,10 @@ async function runWindows (script, { failSilently }) {
         logger.error(`[ipfs on path] ${err.toString()}`)
 
         if (!failSilently) {
-          recoverableErrorDialog(err)
+          recoverableErrorDialog(err, {
+            title: i18n.t('cantAddIpfsToPath.title'),
+            message: i18n.t('cantAddIpfsToPath.message')
+          })
         }
 
         return resolve(false)

src/move-repository-location.js

@@ -69,7 +69,10 @@ module.exports = function ({ stopIpfs, startIpfs }) {
       logger.info(`[move repository] moved from ${currDir} to ${newDir}`)
     } catch (err) {
       logger.error(`[move repository] ${err.toString()}`)
-      return recoverableErrorDialog(err)
+      return recoverableErrorDialog(err, {
+        title: i18n.t('moveRepositoryFailed.title'),
+        message: i18n.t('moveRepositoryFailed.message', { currDir, newDir })
+      })
     }
 
     config.path = newDir