dev: comments

OffchainLabs · Mar 10, 2025 · f09347d · f09347d
1 parent ff6e400
commit f09347d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/packages/arb-token-bridge-ui/src/components/TransactionHistory/helpers.ts b/packages/arb-token-bridge-ui/src/components/TransactionHistory/helpers.ts
@@ -528,7 +528,7 @@ export async function getUpdatedTeleportTransfer(
     ...tx,
     status,
     timestampResolved,
-    l1ToL2MsgData,
+    l1ToL2MsgData, // note: in contrast to general deposits which use `parentToChildMsgData`, Teleport transfers still follow L1/L2/L3 terminology, so we have `l1ToL2MsgData` and `l2ToL3MsgData`
     l2ToL3MsgData
   }
 

diff --git a/packages/arb-token-bridge-ui/src/state/app/state.ts b/packages/arb-token-bridge-ui/src/state/app/state.ts
@@ -69,6 +69,8 @@ export interface MergedTransaction {
 }
 
 export interface TeleporterMergedTransaction extends MergedTransaction {
+  // note: in contrast to general deposits which use `parentToChildMsgData`,
+  // Teleport transfers still follow L1/L2/L3 terminology, so we have `l1ToL2MsgData` and `l2ToL3MsgData`
   l1ToL2MsgData?: ParentToChildMessageData
   l2ToL3MsgData: L2ToL3MessageData
 }

diff --git a/packages/arb-token-bridge-ui/src/state/app/utils.ts b/packages/arb-token-bridge-ui/src/state/app/utils.ts
@@ -63,7 +63,10 @@ export const getDepositStatus = (
   }
 
   if (isTeleportTx(tx)) {
-    const { l2ToL3MsgData, l1ToL2MsgData } = tx
+    // note: in contrast to general deposits which use `parentToChildMsgData`
+    // teleport transfers still follow L1/L2/L3 terminology, so we have `l1ToL2MsgData` and `l2ToL3MsgData`
+
+    const { l1ToL2MsgData, l2ToL3MsgData } = tx
 
     // if any of the retryable info is missing, first fetch might be pending
     if (!l1ToL2MsgData || !l2ToL3MsgData) return DepositStatus.L2_PENDING