From 6bb782677cd14e3ec6af249f5d8a2e93dc8e8b5f Mon Sep 17 00:00:00 2001
From: Sebastian <sebasjm@gmail.com>
Date: Tue, 16 Apr 2024 12:17:32 -0300
Subject: fix 8392

---
 .../src/wallet/Settings.tsx                        |  1 +
 .../src/wallet/Transaction.tsx                     | 97 +++++++++++-----------
 2 files changed, 51 insertions(+), 47 deletions(-)

(limited to 'packages/taler-wallet-webextension/src/wallet')

diff --git a/packages/taler-wallet-webextension/src/wallet/Settings.tsx b/packages/taler-wallet-webextension/src/wallet/Settings.tsx
index 2d80e4c36..0d0a31a2d 100644
--- a/packages/taler-wallet-webextension/src/wallet/Settings.tsx
+++ b/packages/taler-wallet-webextension/src/wallet/Settings.tsx
@@ -274,6 +274,7 @@ function AdvanceSettings(): VNode {
             <Checkbox
               label={label}
               name={name}
+              key={name}
               description={description}
               enabled={settings[settingsName]}
               onToggle={async () => {
diff --git a/packages/taler-wallet-webextension/src/wallet/Transaction.tsx b/packages/taler-wallet-webextension/src/wallet/Transaction.tsx
index af6868434..1f0293352 100644
--- a/packages/taler-wallet-webextension/src/wallet/Transaction.tsx
+++ b/packages/taler-wallet-webextension/src/wallet/Transaction.tsx
@@ -242,53 +242,56 @@ function TransactionTemplate({
             )}
           />
         ) : undefined}
-        {transaction.txState.minor === TransactionMinorState.KycRequired && (
-          <AlertView
-            alert={{
-              type: "warning",
-              message: i18n.str`KYC check required for the transaction to complete.`,
-              description:
-                transaction.kycUrl && typeof transaction.kycUrl === "string" ? (
-                  <div>
-                    <i18n.Translate>
-                      Follow this link to the{` `}
-                      <a
-                        rel="noreferrer"
-                        target="_bank"
-                        href={transaction.kycUrl}
-                      >
-                        KYC verifier.
-                      </a>
-                    </i18n.Translate>
-                  </div>
-                ) : (
-                  i18n.str`No additional information has been provided.`
-                ),
-            }}
-          />
-        )}
-        {transaction.txState.minor === TransactionMinorState.AmlRequired && (
-          <WarningBox>
-            <i18n.Translate>
-              The transaction has been blocked since the account required an AML
-              check.
-            </i18n.Translate>
-          </WarningBox>
-        )}
-        {transaction.txState.major === TransactionMajorState.Pending && (
-          <WarningBox>
-            <div style={{ justifyContent: "center", lineHeight: "25px" }}>
-              <i18n.Translate>This transaction is not completed</i18n.Translate>
-              <Link onClick={onRetry} style={{ padding: 0 }}>
-                <SvgIcon
-                  title={i18n.str`Retry`}
-                  dangerouslySetInnerHTML={{ __html: refreshIcon }}
-                  color="black"
-                />
-              </Link>
-            </div>
-          </WarningBox>
-        )}
+        {transaction.txState.major === TransactionMajorState.Pending &&
+          (transaction.txState.minor === TransactionMinorState.KycRequired ? (
+            <AlertView
+              alert={{
+                type: "warning",
+                message: i18n.str`KYC check required for the transaction to complete.`,
+                description:
+                  transaction.kycUrl &&
+                  typeof transaction.kycUrl === "string" ? (
+                    <div>
+                      <i18n.Translate>
+                        Follow this link to the{` `}
+                        <a
+                          rel="noreferrer"
+                          target="_bank"
+                          href={transaction.kycUrl}
+                        >
+                          KYC verifier.
+                        </a>
+                      </i18n.Translate>
+                    </div>
+                  ) : (
+                    i18n.str`No additional information has been provided.`
+                  ),
+              }}
+            />
+          ) : transaction.txState.minor ===
+            TransactionMinorState.AmlRequired ? (
+            <WarningBox>
+              <i18n.Translate>
+                The transaction has been blocked since the account required an
+                AML check.
+              </i18n.Translate>
+            </WarningBox>
+          ) : (
+            <WarningBox>
+              <div style={{ justifyContent: "center", lineHeight: "25px" }}>
+                <i18n.Translate>
+                  This transaction is not completed
+                </i18n.Translate>
+                <Link onClick={onRetry} style={{ padding: 0 }}>
+                  <SvgIcon
+                    title={i18n.str`Retry`}
+                    dangerouslySetInnerHTML={{ __html: refreshIcon }}
+                    color="black"
+                  />
+                </Link>
+              </div>
+            </WarningBox>
+          ))}
         {transaction.txState.major === TransactionMajorState.Aborted && (
           <InfoBox>
             <i18n.Translate>This transaction was aborted.</i18n.Translate>
-- 
cgit v1.2.3