fix: tenderly simulation uses signer instead of wallet address, fix E…

…xternalLinks
safe-global · Nov 5, 2024 · 4f5cb58 · 4f5cb58
1 parent df91be2
commit 4f5cb58
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/components/common/ExternalLink/index.tsx b/src/components/common/ExternalLink/index.tsx
@@ -22,7 +22,7 @@ const ExternalLink = ({
     </Box>
   )
   return mode === 'link' ? (
-    <Link rel="noreferrer noopener" target="_blank" {...props}>
+    <Link href={href} rel="noreferrer noopener" target="_blank" {...props}>
       {linkContent}
     </Link>
   ) : (

diff --git a/src/components/tx/security/tenderly/index.tsx b/src/components/tx/security/tenderly/index.tsx
@@ -3,7 +3,7 @@ import { useContext, useEffect } from 'react'
 import type { ReactElement } from 'react'
 
 import useSafeInfo from '@/hooks/useSafeInfo'
-import useWallet from '@/hooks/wallets/useWallet'
+import { useSigner } from '@/hooks/wallets/useWallet'
 import CheckIcon from '@/public/images/common/check.svg'
 import CloseIcon from '@/public/images/common/close.svg'
 import { useDarkMode } from '@/hooks/useDarkMode'
@@ -33,7 +33,7 @@ export type TxSimulationProps = {
 // TODO: Test this component
 const TxSimulationBlock = ({ transactions, disabled, gasLimit, executionOwner }: TxSimulationProps): ReactElement => {
   const { safe } = useSafeInfo()
-  const wallet = useWallet()
+  const signer = useSigner()
   const isDarkMode = useDarkMode()
   const { safeTx } = useContext(SafeTxContext)
   const {
@@ -42,13 +42,13 @@ const TxSimulationBlock = ({ transactions, disabled, gasLimit, executionOwner }:
   } = useContext(TxInfoContext)
 
   const handleSimulation = async () => {
-    if (!wallet) {
+    if (!signer) {
       return
     }
 
     simulateTransaction({
       safe,
-      executionOwner: executionOwner ?? wallet.address,
+      executionOwner: executionOwner ?? signer.address,
       transactions,
       gasLimit,
     } as SimulationTxParams)