ST-752: display error messages (#515)

* Display error messages * Improve error parsing * Fix user denied error * fix: typos --------- Co-authored-by: Manuel Alessandro Collazo <[email protected]>
terra-money · Sep 4, 2023 · 7fb207a · 7fb207a
1 parent 5e37cd1
commit 7fb207a
Show file tree

Hide file tree

Showing 2 changed files with 49 additions and 7 deletions.
diff --git a/src/txs/Tx.tsx b/src/txs/Tx.tsx
@@ -36,7 +36,7 @@ import useToPostMultisigTx from "pages/multisig/utils/useToPostMultisigTx"
 import { isWallet, useAuth } from "auth"
 import { PasswordError } from "auth/scripts/keystore"
 
-import { toInput, CoinInput, calcTaxes } from "./utils"
+import { toInput, CoinInput, calcTaxes, parseError } from "./utils"
 import styles from "./Tx.module.scss"
 import { useInterchainLCDClient } from "data/queries/lcdClient"
 import { useInterchainAddresses } from "auth/hooks/useAddress"
@@ -232,7 +232,7 @@ function Tx<TxValues>(props: Props<TxValues>) {
     : props.disabled || ""
 
   const [submitting, setSubmitting] = useState(false)
-  const [error, setError] = useState<Error>()
+  const [error, setError] = useState<any>()
 
   const navigate = useNavigate()
   const toPostMultisigTx = useToPostMultisigTx()
@@ -464,12 +464,10 @@ function Tx<TxValues>(props: Props<TxValues>) {
   const modal = !error
     ? undefined
     : {
-        title: error?.toString().includes("UserDenied")
-          ? t("Transaction was denied by user")
-          : t("Error"),
-        children: error?.toString().includes("UserDenied") ? null : (
+        title: t(parseError(error).title),
+        children: (
           <Pre height={120} normal break>
-            {error?.message}
+            {t(parseError(error).message)}
           </Pre>
         ),
       }

diff --git a/src/txs/utils.ts b/src/txs/utils.ts
@@ -65,3 +65,47 @@ export const calcTaxes = (
       })
   )
 }
+
+export function parseError(error: any): { title: string; message: string } {
+  if (error instanceof Error || typeof error.message === "string") {
+    const { message, name } = error
+
+    switch (message) {
+      case "Ledger device: UNKNOWN_ERROR (0x5515)":
+        return {
+          title: "Ledger error",
+          message:
+            "Please make sure your Ledger device is connected and unlocked.",
+        }
+      case "Unknown Status Code: 28161":
+        return {
+          title: "Ledger error",
+          message:
+            "Please make sure you have the Terra and Cosmos apps installed on your Ledger device.",
+        }
+      case "Failed to execute 'claimInterface' on 'USBDevice': Unable to claim interface.":
+        return {
+          title: "Ledger error",
+          message:
+            "The Ledger device is being used on another tab, please close all the other browser windows and try again.",
+        }
+      default:
+        return {
+          title: name?.includes("Ledger") ? "Ledger error" : "Error",
+          message,
+        }
+    }
+  } else if (typeof error === "string") {
+    return {
+      title: "Error",
+      message: error.includes("User denied")
+        ? "Transaction was denied by user."
+        : error,
+    }
+  } else {
+    return {
+      title: "Error",
+      message: JSON.stringify(error.error ?? error, null, 2),
+    }
+  }
+}