From 2b585b27c0e68fb3e8c5914dda202b7d82b176a2 Mon Sep 17 00:00:00 2001
From: vetalcore <vetalcore@gmail.com>
Date: Mon, 26 Feb 2024 14:43:49 +0200
Subject: [PATCH] fix(extension): resolve sanchonet pr comments

---
 apps/browser-extension-wallet/src/config.ts      |  5 +++--
 .../confirm-transaction/ConfirmTransaction.tsx   |  3 ++-
 .../ConfirmTransactionContent.tsx                |  2 --
 .../DappTransactionContainer.tsx                 |  1 +
 .../VotingProceduresContainer.tsx                |  2 +-
 .../dapp/components/confirm-transaction/hooks.ts |  3 +--
 .../HardForkInitiationActionContainer.tsx        |  2 +-
 .../proposal-procedures/InfoActionContainer.tsx  |  2 +-
 .../NewConstitutionActionContainer.tsx           |  2 +-
 .../NoConfidenceActionContainer.tsx              |  2 +-
 .../ParameterChangeActionContainer.tsx           |  2 +-
 .../TreasuryWithdrawalsActionContainer.tsx       |  2 +-
 .../UpdateCommitteeActionContainer.tsx           |  6 +++---
 .../components/TransactionDetailsProxy.tsx       | 16 +++++-----------
 .../activity/helpers/common-tx-transformer.ts    |  5 +++--
 .../AssetDetailsDrawer/AssetDetails.tsx          | 14 +++-----------
 16 files changed, 28 insertions(+), 41 deletions(-)

diff --git a/apps/browser-extension-wallet/src/config.ts b/apps/browser-extension-wallet/src/config.ts
index 4180e7af3..3e6ad690f 100644
--- a/apps/browser-extension-wallet/src/config.ts
+++ b/apps/browser-extension-wallet/src/config.ts
@@ -22,7 +22,7 @@ export type Config = {
   ADA_PRICE_CHECK_INTERVAL: number;
   TOKEN_PRICE_CHECK_INTERVAL: number;
   AVAILABLE_CHAINS: Wallet.ChainName[];
-  CEXPLORER_BASE_URL: Record<Exclude<EnvironmentTypes, 'Sanchonet'>, string>;
+  CEXPLORER_BASE_URL: Record<EnvironmentTypes, string>;
   CEXPLORER_URL_PATHS: CExplorerUrlPaths;
   SAVED_PRICE_DURATION: number;
 };
@@ -84,7 +84,8 @@ export const config = (): Config => {
     CEXPLORER_BASE_URL: {
       Mainnet: `${process.env.CEXPLORER_URL_MAINNET}`,
       Preprod: `${process.env.CEXPLORER_URL_PREPROD}`,
-      Preview: `${process.env.CEXPLORER_URL_PREVIEW}`
+      Preview: `${process.env.CEXPLORER_URL_PREVIEW}`,
+      Sanchonet: `${process.env.CEXPLORER_URL_SANCHONET}`
     },
     CEXPLORER_URL_PATHS: {
       Tx: 'tx',
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/ConfirmTransaction.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/ConfirmTransaction.tsx
index 696858123..e38ee78ba 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/ConfirmTransaction.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/ConfirmTransaction.tsx
@@ -20,6 +20,7 @@ import { UserPromptService } from '@lib/scripts/background/services';
 import { DAPP_CHANNELS } from '@src/utils/constants';
 import { of, take } from 'rxjs';
 import { runtime } from 'webextension-polyfill';
+import { Skeleton } from 'antd';
 
 export const ConfirmTransaction = (): React.ReactElement => {
   const { t } = useTranslation();
@@ -96,7 +97,7 @@ export const ConfirmTransaction = (): React.ReactElement => {
       pageClassname={styles.spaceBetween}
       title={!confirmTransactionError && txType && t(`core.${txType}.title`)}
     >
-      {req && txType && <ConfirmTransactionContent txType={txType} onError={onError} />}
+      {req && txType ? <ConfirmTransactionContent txType={txType} onError={onError} /> : <Skeleton loading />}
       {!confirmTransactionError && (
         <div className={styles.actions}>
           <Button
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/ConfirmTransactionContent.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/ConfirmTransactionContent.tsx
index cb2129de9..fb1699047 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/ConfirmTransactionContent.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/ConfirmTransactionContent.tsx
@@ -17,8 +17,6 @@ interface Props {
   onError?: () => void;
 }
 
-ConfirmDRepRetirementContainer;
-
 export const ConfirmTransactionContent = ({ txType, onError }: Props): React.ReactElement => {
   const containerPerTypeMap: Record<
     Wallet.Cip30TxType,
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/DappTransactionContainer.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/DappTransactionContainer.tsx
index e6b1a9ae3..1e3d537fc 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/DappTransactionContainer.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/DappTransactionContainer.tsx
@@ -55,6 +55,7 @@ export const DappTransactionContainer = withAddressBookContext(({ errorMessage }
     walletInfo
   });
 
+  // TODO: merge with the upper skeleton check
   if (!txSummary) {
     return <Skeleton />;
   }
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/VotingProceduresContainer.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/VotingProceduresContainer.tsx
index deb5b0a6d..d85526fb1 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/VotingProceduresContainer.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/VotingProceduresContainer.tsx
@@ -45,7 +45,7 @@ export const VotingProceduresContainer = (): React.ReactElement => {
             actionId: {
               index: vote.actionId.actionIndex,
               txHash: vote.actionId.id.toString(),
-              ...(explorerBaseUrl && { txHashUrl: `${explorerBaseUrl}/${vote.actionId.id}` })
+              txHashUrl: `${explorerBaseUrl}/${vote.actionId.id}`
             },
             votingProcedure: {
               vote: t(`core.VotingProcedures.votes.${getVote(vote.votingProcedure.vote)}`),
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/hooks.ts b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/hooks.ts
index fec45b903..63aa3d984 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/hooks.ts
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/hooks.ts
@@ -300,8 +300,7 @@ export const useCexplorerBaseUrl = (): string => {
   const { CEXPLORER_BASE_URL, CEXPLORER_URL_PATHS } = config();
 
   useEffect(() => {
-    const newUrl =
-      environmentName === 'Sanchonet' ? '' : `${CEXPLORER_BASE_URL[environmentName]}/${CEXPLORER_URL_PATHS.Tx}`;
+    const newUrl = `${CEXPLORER_BASE_URL[environmentName]}/${CEXPLORER_URL_PATHS.Tx}`;
     if (newUrl !== explorerBaseUrl) {
       setExplorerBaseUrl(newUrl);
     }
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/HardForkInitiationActionContainer.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/HardForkInitiationActionContainer.tsx
index c24b349aa..c040b672b 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/HardForkInitiationActionContainer.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/HardForkInitiationActionContainer.tsx
@@ -74,7 +74,7 @@ export const HardForkInitiationActionContainer = ({
       anchor: {
         url: anchor.url,
         hash: anchor.dataHash,
-        ...(explorerBaseUrl && { txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}` })
+        txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}`
       }
     },
     protocolVersion: {
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/InfoActionContainer.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/InfoActionContainer.tsx
index 7daa74859..f880b2b86 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/InfoActionContainer.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/InfoActionContainer.tsx
@@ -41,7 +41,7 @@ export const InfoActionContainer = ({ dappInfo, anchor, errorMessage }: Props):
       anchor: {
         url: anchor.url,
         hash: anchor.dataHash,
-        ...(explorerBaseUrl && { txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}` })
+        txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}`
       }
     }
   };
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/NewConstitutionActionContainer.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/NewConstitutionActionContainer.tsx
index ed53a6fe6..aef32fb04 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/NewConstitutionActionContainer.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/NewConstitutionActionContainer.tsx
@@ -77,7 +77,7 @@ export const NewConstitutionActionContainer = ({
       anchor: {
         url: anchor.url,
         hash: anchor.dataHash,
-        ...(explorerBaseUrl && { txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}` })
+        txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}`
       }
     },
     ...(governanceActionId && {
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/NoConfidenceActionContainer.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/NoConfidenceActionContainer.tsx
index 1b9458213..b7abdb9ce 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/NoConfidenceActionContainer.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/NoConfidenceActionContainer.tsx
@@ -69,7 +69,7 @@ export const NoConfidenceActionContainer = ({
       anchor: {
         url: anchor.url,
         hash: anchor.dataHash,
-        ...(explorerBaseUrl && { txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}` })
+        txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}`
       }
     },
     ...(governanceActionId && {
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/ParameterChangeActionContainer.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/ParameterChangeActionContainer.tsx
index 86c6e4b82..d08a5832a 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/ParameterChangeActionContainer.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/ParameterChangeActionContainer.tsx
@@ -266,7 +266,7 @@ export const ParameterChangeActionContainer = ({
     anchor: {
       url: anchor.url,
       hash: anchor.dataHash,
-      ...(explorerBaseUrl && { txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}` })
+      txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}`
     },
     protocolParamUpdate: {
       maxTxExUnits: {
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/TreasuryWithdrawalsActionContainer.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/TreasuryWithdrawalsActionContainer.tsx
index 4f8ff6ebb..4f69c0721 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/TreasuryWithdrawalsActionContainer.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/TreasuryWithdrawalsActionContainer.tsx
@@ -74,7 +74,7 @@ export const TreasuryWithdrawalsActionContainer = ({
       anchor: {
         url: anchor.url,
         hash: anchor.dataHash,
-        ...(explorerBaseUrl && { txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}` })
+        txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}`
       }
     },
     withdrawals: [...withdrawals].map((withdrawal) => ({
diff --git a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/UpdateCommitteeActionContainer.tsx b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/UpdateCommitteeActionContainer.tsx
index ad2a03734..3f35de0b6 100644
--- a/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/UpdateCommitteeActionContainer.tsx
+++ b/apps/browser-extension-wallet/src/features/dapp/components/confirm-transaction/proposal-procedures/UpdateCommitteeActionContainer.tsx
@@ -83,7 +83,7 @@ export const UpdateCommitteeActionContainer = ({
       anchor: {
         url: anchor.url,
         hash: anchor.dataHash,
-        ...(explorerBaseUrl && { txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}` })
+        txHashUrl: `${explorerBaseUrl}/${anchor.dataHash}`
       }
     },
     ...(governanceActionId && {
@@ -94,12 +94,12 @@ export const UpdateCommitteeActionContainer = ({
     }),
     membersToBeAdded: [...membersToBeAdded].map(({ coldCredential: { hash }, epoch }) => ({
       coldCredential: {
-        hash: drepIDasBech32FromHash(hash)
+        hash: hash.toString()
       },
       epoch: epoch.toString()
     })),
     membersToBeRemoved: [...membersToBeRemoved].map(({ hash }) => ({
-      hash: drepIDasBech32FromHash(hash)
+      hash: hash.toString()
     }))
   };
 
diff --git a/apps/browser-extension-wallet/src/views/browser-view/features/activity/components/TransactionDetailsProxy.tsx b/apps/browser-extension-wallet/src/views/browser-view/features/activity/components/TransactionDetailsProxy.tsx
index 4b0e63cc8..616d77ecd 100644
--- a/apps/browser-extension-wallet/src/views/browser-view/features/activity/components/TransactionDetailsProxy.tsx
+++ b/apps/browser-extension-wallet/src/views/browser-view/features/activity/components/TransactionDetailsProxy.tsx
@@ -30,12 +30,10 @@ export const TransactionDetailsProxy = withAddressBookContext(
     const { list: addressList } = useAddressBookContext();
 
     const { CEXPLORER_BASE_URL, CEXPLORER_URL_PATHS } = config();
-    // TODO, remove if sanchonet gets an explorer
-    const explorerBaseUrl = useMemo(() => {
-      if (environmentName === 'Sanchonet') return;
-      // eslint-disable-next-line consistent-return
-      return `${CEXPLORER_BASE_URL[environmentName]}/${CEXPLORER_URL_PATHS.Tx}`;
-    }, [CEXPLORER_BASE_URL, CEXPLORER_URL_PATHS.Tx, environmentName]);
+    const explorerBaseUrl = useMemo(
+      () => `${CEXPLORER_BASE_URL[environmentName]}/${CEXPLORER_URL_PATHS.Tx}`,
+      [CEXPLORER_BASE_URL, CEXPLORER_URL_PATHS.Tx, environmentName]
+    );
     const getHeaderDescription = () => {
       if (activityInfo.type === DelegationActivityType.delegation) return '1 token';
       return ` (${activityInfo?.assetAmount})`;
@@ -78,10 +76,6 @@ export const TransactionDetailsProxy = withAddressBookContext(
       [addressList]
     );
 
-    const environmentSpecificProps = {
-      ...(explorerBaseUrl && { openExternalLink: handleOpenExternalLink })
-    };
-
     return (
       // eslint-disable-next-line react/jsx-pascal-case
       <TransactionDetails
@@ -108,7 +102,7 @@ export const TransactionDetailsProxy = withAddressBookContext(
         proposalProcedures={proposalProcedures}
         votingProcedures={votingProcedures}
         certificates={certificates}
-        {...environmentSpecificProps}
+        openExternalLink={handleOpenExternalLink}
       />
     );
   }
diff --git a/apps/browser-extension-wallet/src/views/browser-view/features/activity/helpers/common-tx-transformer.ts b/apps/browser-extension-wallet/src/views/browser-view/features/activity/helpers/common-tx-transformer.ts
index 0819b7779..be68277a3 100644
--- a/apps/browser-extension-wallet/src/views/browser-view/features/activity/helpers/common-tx-transformer.ts
+++ b/apps/browser-extension-wallet/src/views/browser-view/features/activity/helpers/common-tx-transformer.ts
@@ -23,6 +23,7 @@ import dayjs from 'dayjs';
 import isEmpty from 'lodash/isEmpty';
 import { PriceResult } from '@hooks';
 import { formatPercentages } from '@lace/common';
+import { depositPaidWithSymbol } from '@src/features/dapp/components/confirm-transaction/utils';
 
 const { util, GovernanceActionType, PlutusLanguageVersion, CertificateType } = Wallet.Cardano;
 
@@ -664,11 +665,11 @@ export const governanceProposalsTransformer = ({
             },
             {
               title: 'govActionDeposit',
-              details: [`${governanceActionDeposit?.toString()} ${cardanoCoin.symbol}`]
+              details: [depositPaidWithSymbol(BigInt(governanceActionDeposit), cardanoCoin)]
             },
             {
               title: 'drepDeposit',
-              details: [`${dRepDeposit?.toString()}  ${cardanoCoin.symbol}`]
+              details: [depositPaidWithSymbol(BigInt(dRepDeposit), cardanoCoin)]
             },
             {
               title: 'drepActivity',
diff --git a/apps/browser-extension-wallet/src/views/browser-view/features/assets/components/AssetDetailsDrawer/AssetDetails.tsx b/apps/browser-extension-wallet/src/views/browser-view/features/assets/components/AssetDetailsDrawer/AssetDetails.tsx
index 7c2e0374b..beac455ca 100644
--- a/apps/browser-extension-wallet/src/views/browser-view/features/assets/components/AssetDetailsDrawer/AssetDetails.tsx
+++ b/apps/browser-extension-wallet/src/views/browser-view/features/assets/components/AssetDetailsDrawer/AssetDetails.tsx
@@ -48,11 +48,7 @@ export const AssetDetails = ({
   const { environmentName } = useWalletStore();
   const openExternalLink = useExternalLinkOpener();
 
-  const explorerBaseUrl = useMemo(() => {
-    if (environmentName === 'Sanchonet') return '';
-    // eslint-disable-next-line consistent-return
-    return CEXPLORER_BASE_URL[environmentName];
-  }, [environmentName]);
+  const explorerBaseUrl = useMemo(() => CEXPLORER_BASE_URL[environmentName], [environmentName]);
   const isTxListLoading = activityListStatus === StateStatus.IDLE || activityListStatus === StateStatus.LOADING;
 
   return (
@@ -115,17 +111,13 @@ export const AssetDetails = ({
                     name: t('browserView.assetDetails.fingerprint'),
                     value: fingerprint,
                     showCopyIcon: true,
-                    ...(explorerBaseUrl && {
-                      onClick: () => openExternalLink(`${explorerBaseUrl}/${CEXPLORER_URL_PATHS.Asset}/${fingerprint}`)
-                    })
+                    onClick: () => openExternalLink(`${explorerBaseUrl}/${CEXPLORER_URL_PATHS.Asset}/${fingerprint}`)
                   },
                   {
                     name: t('browserView.assetDetails.policyId'),
                     value: policyId,
                     showCopyIcon: true,
-                    ...(explorerBaseUrl && {
-                      onClick: () => openExternalLink(`${explorerBaseUrl}/${CEXPLORER_URL_PATHS.Policy}/${policyId}`)
-                    })
+                    onClick: () => openExternalLink(`${explorerBaseUrl}/${CEXPLORER_URL_PATHS.Policy}/${policyId}`)
                   }
                 ]}
               />