fix: render issue on detail view for transfers with execution propert…

…y as null (#182)
sprintertech · Jul 5, 2024 · 040c9c3 · 040c9c3
1 parent 7cc589f
commit 040c9c3
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 18 deletions.
diff --git a/src/pages/DetailView/DetailView.tsx b/src/pages/DetailView/DetailView.tsx
@@ -113,7 +113,7 @@ export default function DetailView() {
           <span className={classes.detailsInnerContentTitle}>Source transaction hash:</span>
           <span className={classes.detailsInnerContent}>
             <Link
-              to={txHashLinks(fromDomainInfo!.type, transfer?.deposit?.txHash!, fromDomainExplorerUrl!.url)}
+              to={txHashLinks(fromDomainInfo!.type, fromDomainExplorerUrl!.url, transfer?.deposit?.txHash!)}
               style={{ color: "black" }}
               target="_blank"
             >
@@ -139,26 +139,30 @@ export default function DetailView() {
           <span className={classes.detailsInnerContentTitle}>Destination transaction hash:</span>
           <span className={classes.detailsInnerContent}>
             <Link
-              to={txHashLinks(toDomainInfo!.type, transfer?.execution?.txHash!, toDomainExplorerUrl!.url)}
+              to={txHashLinks(toDomainInfo!.type, toDomainExplorerUrl!.url, transfer?.execution?.txHash!)}
               style={{ color: "black" }}
               target="_blank"
             >
               {transfer?.execution && transfer?.execution?.txHash}
             </Link>
-            <span
-              className={classes.copyIcon}
-              onClick={() => {
-                void navigator.clipboard?.writeText(transfer?.execution?.txHash!)
-                dispatcher({
-                  type: "set_clipboard_message_t2",
-                  payload: "Copied to clipboard!",
-                })
-              }}
-            >
-              <Tooltip title={state.clipboardMessageT2} placement="top" arrow>
-                <ContentCopyIcon fontSize="small" />
-              </Tooltip>
-            </span>
+            {transfer?.execution ? (
+              <span
+                className={classes.copyIcon}
+                onClick={() => {
+                  void navigator.clipboard?.writeText(transfer?.execution?.txHash!)
+                  dispatcher({
+                    type: "set_clipboard_message_t2",
+                    payload: "Copied to clipboard!",
+                  })
+                }}
+              >
+                <Tooltip title={state.clipboardMessageT2} placement="top" arrow>
+                  <ContentCopyIcon fontSize="small" />
+                </Tooltip>
+              </span>
+            ) : (
+              <span>Pending proposal execution</span>
+            )}
           </span>
         </div>
         <div className={classes.detailsContainer}>

diff --git a/src/utils/Helpers.tsx b/src/utils/Helpers.tsx
@@ -306,12 +306,12 @@ export const renderFormatedConvertedAmount = (type: ResourceTypes, usdValue: num
   return ""
 }
 
-export const txHashLinks = (type: DomainTypes, txHash: string, domainExplorerUrl: string): string => {
+export const txHashLinks = (type: DomainTypes, domainExplorerUrl: string, txHash?: string): string => {
   switch (type) {
     case DomainTypes.EVM:
       return `${domainExplorerUrl}/tx/${txHash}`
     case DomainTypes.SUBSTRATE:
-      return `${domainExplorerUrl}/${txHash.split("-")[0]}`
+      return `${domainExplorerUrl}/${txHash?.split("-")[0] || ""}`
     default:
       return ""
   }