taler-typescript-core

commit 9bd94fc648359ef635138d518f3cc9515ba806f3
parent e050dfd8a87352944553cb8b0594bc7b92011e71
Author: Sebastian <sebasjm@gmail.com>
Date:   Wed,  1 Nov 2023 23:17:33 -0300

show error inplace instead of notification

Diffstat:
M
packages/demobank-ui/src/pages/WithdrawalOperationPage.tsx
 | 
14
++++----------

1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/packages/demobank-ui/src/pages/WithdrawalOperationPage.tsx b/packages/demobank-ui/src/pages/WithdrawalOperationPage.tsx
@@ -16,16 +16,14 @@
 
 import {
   Logger,
-  TranslatedString,
   parseWithdrawUri,
   stringifyWithdrawUri
 } from "@gnu-taler/taler-util";
 import {
-  notifyError,
   useTranslationContext
 } from "@gnu-taler/web-util/browser";
 import { Fragment, VNode, h } from "preact";
-import { Loading } from "../components/Loading.js";
+import { Attention } from "../components/Attention.js";
 import { useBankCoreApiContext } from "../context/config.js";
 import { useSettings } from "../hooks/settings.js";
 import { WithdrawalQRCode } from "./WithdrawalQRCode.js";
@@ -39,8 +37,6 @@ export function WithdrawalOperationPage({
   operationId: string;
   onContinue: () => void;
 }): VNode {
-  //FIXME: libeufin sandbox should return show to create the integration api endpoint
-  //or return withdrawal uri from response
   const { api } = useBankCoreApiContext()
   const uri = stringifyWithdrawUri({
     bankIntegrationApiBaseUrl: api.getIntegrationAPI().baseUrl,
@@ -51,11 +47,9 @@ export function WithdrawalOperationPage({
   const [settings, updateSettings] = useSettings();
 
   if (!parsedUri) {
-    notifyError(
-      i18n.str`The Withdrawal URI is not valid`,
-      uri as TranslatedString
-    );
-    return <Loading />;
+    return <Attention type="danger" title={i18n.str`The Withdrawal URI is not valid`}>
+      {uri}
+    </Attention>
   }
 
   return (