add error messages for new error cases

13 files changed, 108 insertions, 55 deletions

diff --git a/packages/demobank-ui/src/pages/AccountPage/state.ts b/packages/demobank-ui/src/pages/AccountPage/state.ts
index 6da066d77..d28aba7bf 100644
--- a/packages/demobank-ui/src/pages/AccountPage/state.ts
+++ b/packages/demobank-ui/src/pages/AccountPage/state.ts
@@ -48,11 +48,6 @@ export function useComponentState({ account, goToConfirmOperation }: Props): Sta
         status: "login",
         reason: "not-found",
       }
-      case "no-rights": return {
-        //users are forbiden to see others account
-        status: "login",
-        reason: "not-found",
-      }
       default: {
         assertUnreachable(result)
       }
diff --git a/packages/demobank-ui/src/pages/OperationState/views.tsx b/packages/demobank-ui/src/pages/OperationState/views.tsx
index ea38525b9..e623b0dc2 100644
--- a/packages/demobank-ui/src/pages/OperationState/views.tsx
+++ b/packages/demobank-ui/src/pages/OperationState/views.tsx
@@ -129,6 +129,12 @@ export function NeedConfirmationView({ error, onAbort: doAbort, onConfirm: doCon
           description: hasError.detail.hint as TranslatedString,
           debug: hasError.detail,
         });
+        case "insufficient-funds": return notify({
+          type: "error",
+          title: i18n.str`Your balance is not enough.`,
+          description: hasError.detail.hint as TranslatedString,
+          debug: hasError.detail,
+        });
         default: assertUnreachable(hasError)
       }
     })
diff --git a/packages/demobank-ui/src/pages/PaytoWireTransferForm.tsx b/packages/demobank-ui/src/pages/PaytoWireTransferForm.tsx
index 97e38d75e..31592039f 100644
--- a/packages/demobank-ui/src/pages/PaytoWireTransferForm.tsx
+++ b/packages/demobank-ui/src/pages/PaytoWireTransferForm.tsx
@@ -160,12 +160,30 @@ export function PaytoWireTransferForm({
             description: res.detail.hint as TranslatedString,
             debug: res.detail,
           })
-          case "account-not-found": return notify({
+          case "creditor-not-found": return notify({
             type: "error",
             title: i18n.str`The destination account "${puri}" was not found.`,
             description: res.detail.hint as TranslatedString,
             debug: res.detail,
           })
+          case "creditor-same": return notify({
+            type: "error",
+            title: i18n.str`The origin and the destination of the transfer can't be the same.`,
+            description: res.detail.hint as TranslatedString,
+            debug: res.detail,
+          })
+          case "insufficient-funds": return notify({
+            type: "error",
+            title: i18n.str`Your balance is not enough.`,
+            description: res.detail.hint as TranslatedString,
+            debug: res.detail,
+          })
+          case "not-found": return notify({
+            type: "error",
+            title: i18n.str`The origin account "${puri}" was not found.`,
+            description: res.detail.hint as TranslatedString,
+            debug: res.detail,
+          })
           default: assertUnreachable(res)
         }
       }
diff --git a/packages/demobank-ui/src/pages/RegistrationPage.tsx b/packages/demobank-ui/src/pages/RegistrationPage.tsx
index fdf2c0e9d..c2eca25e8 100644
--- a/packages/demobank-ui/src/pages/RegistrationPage.tsx
+++ b/packages/demobank-ui/src/pages/RegistrationPage.tsx
@@ -99,15 +99,15 @@ function RegistrationForm({ onComplete, onCancel }: { onComplete: () => void, on
       const creationResponse = await api.createAccount("" as AccessToken, { name: name ?? "", username, password });
       if (creationResponse.type === "fail") {
         switch (creationResponse.case) {
-          case "invalid-input": return notify({
+          case "invalid-phone-or-email": return notify({
             type: "error",
-            title: i18n.str`Some of the input fields are invalid.`,
+            title: i18n.str`Server replied with invalid phone or email.`,
             description: creationResponse.detail.hint as TranslatedString,
             debug: creationResponse.detail,
           })
-          case "unable-to-create": return notify({
+          case "insufficient-funds": return notify({
             type: "error",
-            title: i18n.str`Unable to create that account.`,
+            title: i18n.str`Registration is disabled because the bank ran out of bonus credit.`,
             description: creationResponse.detail.hint as TranslatedString,
             debug: creationResponse.detail,
           })
@@ -117,9 +117,21 @@ function RegistrationForm({ onComplete, onCancel }: { onComplete: () => void, on
             description: creationResponse.detail.hint as TranslatedString,
             debug: creationResponse.detail,
           })
-          case "already-exist": return notify({
+          case "payto-already-exists": return notify({
             type: "error",
-            title: i18n.str`That username is already taken`,
+            title: i18n.str`That account id is already taken.`,
+            description: creationResponse.detail.hint as TranslatedString,
+            debug: creationResponse.detail,
+          })
+          case "username-already-exists": return notify({
+            type: "error",
+            title: i18n.str`That username is already taken.`,
+            description: creationResponse.detail.hint as TranslatedString,
+            debug: creationResponse.detail,
+          })
+          case "username-reserved": return notify({
+            type: "error",
+            title: i18n.str`That username can't be used because is reserved.`,
             description: creationResponse.detail.hint as TranslatedString,
             debug: creationResponse.detail,
           })
diff --git a/packages/demobank-ui/src/pages/ShowAccountDetails.tsx b/packages/demobank-ui/src/pages/ShowAccountDetails.tsx
index eb8ea8f20..c07802273 100644
--- a/packages/demobank-ui/src/pages/ShowAccountDetails.tsx
+++ b/packages/demobank-ui/src/pages/ShowAccountDetails.tsx
@@ -45,7 +45,6 @@ export function ShowAccountDetails({
     switch (result.case) {
       case "not-found": return <LoginForm reason="not-found" />
       case "unauthorized": return <LoginForm reason="forbidden" />
-      case "no-rights": return <LoginForm reason="forbidden" />
       default: assertUnreachable(result)
     }
   }
@@ -95,7 +94,7 @@ export function ShowAccountDetails({
 
   return (
     <Fragment>
-          <ShowLocalNotification notification={notification} />
+      <ShowLocalNotification notification={notification} />
       {accountIsTheCurrentUser ?
         <ProfileNavigation current="details" />
         :
diff --git a/packages/demobank-ui/src/pages/WithdrawalConfirmationQuestion.tsx b/packages/demobank-ui/src/pages/WithdrawalConfirmationQuestion.tsx
index 89538e305..b548c0d16 100644
--- a/packages/demobank-ui/src/pages/WithdrawalConfirmationQuestion.tsx
+++ b/packages/demobank-ui/src/pages/WithdrawalConfirmationQuestion.tsx
@@ -119,6 +119,12 @@ export function WithdrawalConfirmationQuestion({
             description: resp.detail.hint as TranslatedString,
             debug: resp.detail,
           })
+          case "insufficient-funds": return notify({
+            type: "error",
+            title: i18n.str`Your balance is not enough for the operation.`,
+            description: resp.detail.hint as TranslatedString,
+            debug: resp.detail,
+          })
           default: assertUnreachable(resp)
         }
       }
@@ -161,7 +167,7 @@ export function WithdrawalConfirmationQuestion({
 
   return (
     <Fragment>
-          <ShowLocalNotification notification={notification} />
+      <ShowLocalNotification notification={notification} />
 
       <div class="bg-white shadow sm:rounded-lg">
         <div class="px-4 py-5 sm:p-6">
diff --git a/packages/demobank-ui/src/pages/admin/Account.tsx b/packages/demobank-ui/src/pages/admin/Account.tsx
index 7109b082f..19189bec4 100644
--- a/packages/demobank-ui/src/pages/admin/Account.tsx
+++ b/packages/demobank-ui/src/pages/admin/Account.tsx
@@ -23,9 +23,8 @@ export function WireTransfer({ toAccount, onRegister, onCancel, onSuccess }: { o
   }
   if (result.type === "fail") {
     switch (result.case) {
-      case "unauthorized": return <LoginForm reason="forbidden"  />
+      case "unauthorized": return <LoginForm reason="forbidden" />
       case "not-found": return <LoginForm reason="not-found" />
-      case "no-rights": return <LoginForm reason="not-found" />
       default: assertUnreachable(result)
     }
   }
diff --git a/packages/demobank-ui/src/pages/admin/AccountList.tsx b/packages/demobank-ui/src/pages/admin/AccountList.tsx
index 056035750..be5194e6d 100644
--- a/packages/demobank-ui/src/pages/admin/AccountList.tsx
+++ b/packages/demobank-ui/src/pages/admin/AccountList.tsx
@@ -30,9 +30,8 @@ export function AccountList({ onRemoveAccount, onShowAccountDetails, onUpdateAcc
   }
   if (result.data.type === "fail") {
     switch (result.data.case) {
-      case "unauthorized": return <Fragment/>
-      case "no-rights": return <Fragment/>
-      default: assertUnreachable(result.data)
+      case "unauthorized": return <Fragment />
+      default: assertUnreachable(result.data.case)
     }
   }
 
diff --git a/packages/demobank-ui/src/pages/admin/AdminHome.tsx b/packages/demobank-ui/src/pages/admin/AdminHome.tsx
index 1e3d3d748..a30cae547 100644
--- a/packages/demobank-ui/src/pages/admin/AdminHome.tsx
+++ b/packages/demobank-ui/src/pages/admin/AdminHome.tsx
@@ -124,25 +124,28 @@ function Metrics(): VNode {
     </div>
     <dl class="mt-5 grid grid-cols-1 divide-y divide-gray-200 overflow-hidden rounded-lg bg-white shadow-lg md:grid-cols-3 md:divide-x md:divide-y-0">
 
-      <div class="px-4 py-5 sm:p-6">
-        <dt class="text-base font-normal text-gray-900">
-          <i18n.Translate>Cashin</i18n.Translate>
-        </dt>
-        <MetricValue
-          current={resp.current.body.cashinExternalVolume}
-          previous={resp.previous.body.cashinExternalVolume}
-        />
-      </div>
-
-      <div class="px-4 py-5 sm:p-6">
-        <dt class="text-base font-normal text-gray-900">
-          <i18n.Translate>Cashout</i18n.Translate>
-        </dt>
-        <MetricValue
-          current={resp.current.body.cashoutExternalVolume}
-          previous={resp.previous.body.cashoutExternalVolume}
-        />
-      </div>
+      {resp.current.body.type !== "with-cashout" || resp.previous.body.type !== "with-cashout" ? undefined :
+        <Fragment>
+          <div class="px-4 py-5 sm:p-6">
+            <dt class="text-base font-normal text-gray-900">
+              <i18n.Translate>Cashin</i18n.Translate>
+            </dt>
+            <MetricValue
+              current={resp.current.body.cashinExternalVolume}
+              previous={resp.previous.body.cashinExternalVolume}
+            />
+          </div>
+          <div class="px-4 py-5 sm:p-6">
+            <dt class="text-base font-normal text-gray-900">
+              <i18n.Translate>Cashout</i18n.Translate>
+            </dt>
+            <MetricValue
+              current={resp.current.body.cashoutExternalVolume}
+              previous={resp.previous.body.cashoutExternalVolume}
+            />
+          </div>
+        </Fragment>
+      }
       <div class="px-4 py-5 sm:p-6">
         <dt class="text-base font-normal text-gray-900">
           <i18n.Translate>Payout</i18n.Translate>
diff --git a/packages/demobank-ui/src/pages/admin/CreateNewAccount.tsx b/packages/demobank-ui/src/pages/admin/CreateNewAccount.tsx
index f2c1d5456..3f4364c16 100644
--- a/packages/demobank-ui/src/pages/admin/CreateNewAccount.tsx
+++ b/packages/demobank-ui/src/pages/admin/CreateNewAccount.tsx
@@ -49,27 +49,39 @@ export function CreateNewAccount({
         onCreateSuccess();
       } else {
         switch (resp.case) {
-          case "invalid-input": return notify({
+          case "invalid-phone-or-email": return notify({
             type: "error",
-            title: i18n.str`Server replied that input data was invalid`,
+            title: i18n.str`Server replied with invalid phone or email`,
             description: resp.detail.hint as TranslatedString,
             debug: resp.detail,
           })
-          case "unable-to-create": return notify({
+          case "unauthorized": return notify({
             type: "error",
-            title: i18n.str`The account name is registered.`,
+            title: i18n.str`The rights to perform the operation are not sufficient`,
             description: resp.detail.hint as TranslatedString,
             debug: resp.detail,
           })
-          case "unauthorized": return notify({
+          case "username-already-exists": return notify({
             type: "error",
-            title: i18n.str`The rights to perform the operation are not sufficient`,
+            title: i18n.str`Account username is already taken`,
+            description: resp.detail.hint as TranslatedString,
+            debug: resp.detail,
+          })
+          case "payto-already-exists": return notify({
+            type: "error",
+            title: i18n.str`Account id is already taken`,
+            description: resp.detail.hint as TranslatedString,
+            debug: resp.detail,
+          })
+          case "insufficient-funds": return notify({
+            type: "error",
+            title: i18n.str`Bank ran out of bonus credit.`,
             description: resp.detail.hint as TranslatedString,
             debug: resp.detail,
           })
-          case "already-exist": return notify({
+          case "username-reserved": return notify({
             type: "error",
-            title: i18n.str`Account name is already taken`,
+            title: i18n.str`Account username can't be used because is reserved`,
             description: resp.detail.hint as TranslatedString,
             debug: resp.detail,
           })
diff --git a/packages/demobank-ui/src/pages/admin/RemoveAccount.tsx b/packages/demobank-ui/src/pages/admin/RemoveAccount.tsx
index 1a5255595..fa9693941 100644
--- a/packages/demobank-ui/src/pages/admin/RemoveAccount.tsx
+++ b/packages/demobank-ui/src/pages/admin/RemoveAccount.tsx
@@ -45,7 +45,6 @@ export function RemoveAccount({
     switch (result.case) {
       case "unauthorized": return <LoginForm reason="forbidden" />
       case "not-found": return <LoginForm reason="not-found" />
-      case "no-rights": return <LoginForm reason="not-found" />
       default: assertUnreachable(result)
     }
   }
@@ -82,9 +81,9 @@ export function RemoveAccount({
             description: resp.detail.hint as TranslatedString,
             debug: resp.detail,
           })
-          case "unable-to-delete": return notify({
+          case "username-reserved": return notify({
             type: "error",
-            title: i18n.str`The administrator specified a institutional username.`,
+            title: i18n.str`Can't delete a reserved username.`,
             description: resp.detail.hint as TranslatedString,
             debug: resp.detail,
           })
diff --git a/packages/demobank-ui/src/pages/business/CreateCashout.tsx b/packages/demobank-ui/src/pages/business/CreateCashout.tsx
index 8d90e9205..5c284be24 100644
--- a/packages/demobank-ui/src/pages/business/CreateCashout.tsx
+++ b/packages/demobank-ui/src/pages/business/CreateCashout.tsx
@@ -100,7 +100,6 @@ export function CreateCashout({
     switch (resultAccount.case) {
       case "unauthorized": return <LoginForm reason="forbidden" />
       case "not-found": return <LoginForm reason="not-found" />
-      case "no-rights": return <LoginForm reason="not-found" />
       default: assertUnreachable(resultAccount)
     }
   }
@@ -178,7 +177,7 @@ export function CreateCashout({
 
   return (
     <div>
-          <ShowLocalNotification notification={notification} />
+      <ShowLocalNotification notification={notification} />
       <h1>New cashout</h1>
       <form class="pure-form">
         <fieldset>
@@ -393,9 +392,15 @@ export function CreateCashout({
                       description: resp.detail.hint as TranslatedString,
                       debug: resp.detail,
                     });
-                    case "cashout-or-tan-not-supported": return notify({
+                    case "account-not-found": return notify({
                       type: "error",
-                      title: i18n.str`The bank does not support the TAN channel for this operation`,
+                      title: i18n.str`Account not found`,
+                      description: resp.detail.hint as TranslatedString,
+                      debug: resp.detail,
+                    });
+                    case "cashout-not-supported": return notify({
+                      type: "error",
+                      title: i18n.str`The bank does not support cashout`,
                       description: resp.detail.hint as TranslatedString,
                       debug: resp.detail,
                     });
diff --git a/packages/demobank-ui/src/pages/business/ShowCashoutDetails.tsx b/packages/demobank-ui/src/pages/business/ShowCashoutDetails.tsx
index 7e7ed21cb..a8b57b90c 100644
--- a/packages/demobank-ui/src/pages/business/ShowCashoutDetails.tsx
+++ b/packages/demobank-ui/src/pages/business/ShowCashoutDetails.tsx
@@ -65,7 +65,7 @@ export function ShowCashoutDetails({
   }
   if (result.type === "fail") {
     switch (result.case) {
-      case "already-aborted": return <Attention type="warning" title={i18n.str`This cashout has already been aborted.`}>
+      case "not-found": return <Attention type="warning" title={i18n.str`This cashout not found. Maybe already aborted.`}>
       </Attention>
       case "cashout-not-supported": return <Attention type="warning" title={i18n.str`Cashouts are not supported`}>
       </Attention>