From f969d5188594443c0d6ddf235b9b0e41cba59c9a Mon Sep 17 00:00:00 2001
From: Nelito Junior <ngvjqp@gmail.com>
Date: Mon, 3 Feb 2025 18:07:14 -0300
Subject: [PATCH] refactor: rename transaction props for clarity and
 consistency

---
 .../TransactionRequest/TransactionRequest.tsx      |  6 +++---
 .../Transaction/components/TxDetails/TxDetails.tsx | 13 +++++++------
 .../Transaction/hooks/useSimplifiedTransaction.tsx | 14 +++++++-------
 .../Transaction/pages/TxApprove/TxApprove.tsx      |  4 ++--
 .../systems/Transaction/pages/TxView/TxView.tsx    |  2 +-
 5 files changed, 20 insertions(+), 19 deletions(-)

diff --git a/packages/app/src/systems/DApp/pages/TransactionRequest/TransactionRequest.tsx b/packages/app/src/systems/DApp/pages/TransactionRequest/TransactionRequest.tsx
index bf041624a..22870fb71 100644
--- a/packages/app/src/systems/DApp/pages/TransactionRequest/TransactionRequest.tsx
+++ b/packages/app/src/systems/DApp/pages/TransactionRequest/TransactionRequest.tsx
@@ -73,15 +73,15 @@ export function TransactionRequest() {
         <Layout.Content css={styles.content}>
           {shouldShowTxSimulated && (
             <TxDetails
-              summary={txSummarySimulated}
-              request={proposedTxRequest}
+              tx={txSummarySimulated}
+              txRequest={proposedTxRequest}
               showDetails
               isLoading={isLoadingInfo || !defaultValues}
             />
           )}
           {shouldShowTxExecuted && (
             <TxDetails
-              summary={txSummaryExecuted}
+              tx={txSummaryExecuted}
               showDetails
               footer={
                 status('failed') && (
diff --git a/packages/app/src/systems/Transaction/components/TxDetails/TxDetails.tsx b/packages/app/src/systems/Transaction/components/TxDetails/TxDetails.tsx
index 0e6ab79c2..6e93df42f 100644
--- a/packages/app/src/systems/Transaction/components/TxDetails/TxDetails.tsx
+++ b/packages/app/src/systems/Transaction/components/TxDetails/TxDetails.tsx
@@ -17,8 +17,8 @@ import { TxHeaderSimple } from './TxHeaderSimple';
 export type TxViewVariant = 'default' | 'history';
 
 type TxDetailsProps = {
-  summary?: TransactionSummary | TransactionResult;
-  request?: TransactionRequest;
+  tx?: TransactionSummary | TransactionResult;
+  txRequest?: TransactionRequest;
   showDetails?: boolean;
   isLoading?: boolean;
   footer?: React.ReactNode;
@@ -26,20 +26,21 @@ type TxDetailsProps = {
 };
 
 export function TxDetails({
-  summary,
-  request,
+  tx,
+  txRequest,
   showDetails = true,
   isLoading: externalLoading,
   footer,
   variant = 'default',
 }: TxDetailsProps) {
+  // TODO: Handle errors.simulateTxErrors, isConfirm, fees
   const [isCustomFees, setIsCustomFees] = useState(false);
   const [_, setSelectedTip] = useState<BN>();
   const isHistory = variant === 'history';
 
   const { transaction, isReady } = useSimplifiedTransaction({
-    summary,
-    request,
+    tx,
+    txRequest,
   });
 
   if (!isReady || !transaction) return null;
diff --git a/packages/app/src/systems/Transaction/hooks/useSimplifiedTransaction.tsx b/packages/app/src/systems/Transaction/hooks/useSimplifiedTransaction.tsx
index 8e318cabc..e1fdfd0d2 100644
--- a/packages/app/src/systems/Transaction/hooks/useSimplifiedTransaction.tsx
+++ b/packages/app/src/systems/Transaction/hooks/useSimplifiedTransaction.tsx
@@ -5,21 +5,21 @@ import type { SimplifiedTransaction } from '../types';
 import { simplifyTransaction } from '../utils/simplifyTransaction';
 
 type UseSimplifiedTransactionProps = {
-  summary?: TransactionSummary;
-  request?: TransactionRequest;
+  tx?: TransactionSummary;
+  txRequest?: TransactionRequest;
 };
 
 export function useSimplifiedTransaction({
-  summary,
-  request,
+  tx,
+  txRequest,
 }: UseSimplifiedTransactionProps) {
   const { account } = useAccounts();
 
   const transaction = useMemo<SimplifiedTransaction | undefined>(() => {
-    if (!summary) return undefined;
+    if (!tx) return undefined;
 
-    return simplifyTransaction(summary, request, account?.address);
-  }, [summary, request, account?.address]);
+    return simplifyTransaction(tx, txRequest, account?.address);
+  }, [tx, txRequest, account?.address]);
 
   return {
     transaction,
diff --git a/packages/app/src/systems/Transaction/pages/TxApprove/TxApprove.tsx b/packages/app/src/systems/Transaction/pages/TxApprove/TxApprove.tsx
index dd64b3281..9a71e55ac 100644
--- a/packages/app/src/systems/Transaction/pages/TxApprove/TxApprove.tsx
+++ b/packages/app/src/systems/Transaction/pages/TxApprove/TxApprove.tsx
@@ -31,7 +31,7 @@ export const TxApprove = () => {
       <Dialog.Description as="div" css={styles.description}>
         {ctx.shouldShowTxSimulated && (
           <TxDetails
-            summary={ctx.txSummarySimulated}
+            tx={ctx.txSummarySimulated}
             showDetails
             isLoading={!ctx.txSummarySimulated}
             variant="default"
@@ -51,7 +51,7 @@ export const TxApprove = () => {
         )}
         {ctx.shouldShowTxExecuted && (
           <TxDetails
-            summary={ctx.txSummaryExecuted}
+            tx={ctx.txSummaryExecuted}
             showDetails
             isLoading={false}
             variant="default"
diff --git a/packages/app/src/systems/Transaction/pages/TxView/TxView.tsx b/packages/app/src/systems/Transaction/pages/TxView/TxView.tsx
index fa8723346..2023da4b9 100644
--- a/packages/app/src/systems/Transaction/pages/TxView/TxView.tsx
+++ b/packages/app/src/systems/Transaction/pages/TxView/TxView.tsx
@@ -28,7 +28,7 @@ export function TxView() {
         )}
         {txResult && (
           <TxDetails
-            summary={txResult}
+            tx={txResult}
             showDetails={ctx.shouldShowTxFee}
             isLoading={!txResult}
             variant="history"