more adjustments around parsePrefixedAddress

gnosisguild · Dec 11, 2024 · c79c854 · c79c854
1 parent a49f2ca
commit c79c854
Show file tree

Hide file tree

Showing 7 changed files with 9 additions and 10 deletions.
diff --git a/extension/src/panel/execution-routes/useRouteConnect.ts b/extension/src/panel/execution-routes/useRouteConnect.ts
@@ -63,7 +63,7 @@ const useCanEstablishConnection = (route: ExecutionRoute) => {
 
   const pilotAddress =
     route.initiator && route.initiator !== `eoa:` + ZeroAddress
-      ? parsePrefixedAddress(route.initiator)[1].toLowerCase()
+      ? parsePrefixedAddress(route.initiator).toLowerCase()
       : undefined
 
   if (pilotAddress == null) {

diff --git a/extension/src/panel/pages/legacyConnectionMigrations.ts b/extension/src/panel/pages/legacyConnectionMigrations.ts
@@ -141,8 +141,7 @@ export function asLegacyConnection(route: ExecutionRoute): LegacyConnection {
   invariant(chainId != null, 'chainId is empty')
 
   const pilotAddress =
-    (route.initiator &&
-      parsePrefixedAddress(route.initiator)[1].toLowerCase()) ||
+    (route.initiator && parsePrefixedAddress(route.initiator).toLowerCase()) ||
     ''
 
   const moduleWaypoint = route.waypoints?.find(

diff --git a/extension/src/panel/pages/routes/edit.$routeId/wallet/ConnectWallet.tsx b/extension/src/panel/pages/routes/edit.$routeId/wallet/ConnectWallet.tsx
@@ -96,7 +96,7 @@ const getPilotAddress = (route: ExecutionRoute) => {
     return null
   }
 
-  const address = parsePrefixedAddress(route.initiator)[1].toLowerCase()
+  const address = parsePrefixedAddress(route.initiator).toLowerCase()
 
   if (address === ZeroAddress) {
     return null

diff --git a/extension/src/panel/pages/transactions/RolePermissionCheck.tsx b/extension/src/panel/pages/transactions/RolePermissionCheck.tsx
@@ -40,7 +40,7 @@ const simulateRolesTransaction = async (
     throw new Error('Only transaction execution is currently supported')
   }
 
-  const [, from] = parsePrefixedAddress(plan[0].from)
+  const from = parsePrefixedAddress(plan[0].from)
   const tx = {
     ...plan[0].transaction,
     from,

diff --git a/extension/src/panel/pages/transactions/Submit.tsx b/extension/src/panel/pages/transactions/Submit.tsx
@@ -107,7 +107,7 @@ export const Submit = () => {
       console.debug(
         `Transaction batch has been proposed with safeTxHash ${safeTxHash}`,
       )
-      const [, avatarAddress] = parsePrefixedAddress(avatar)
+      const avatarAddress = parsePrefixedAddress(avatar)
       successToast({
         title: 'Transaction batch has been proposed for execution',
         message: (

diff --git a/extension/src/panel/providers-ui/ProvideProvider.tsx b/extension/src/panel/providers-ui/ProvideProvider.tsx
@@ -28,17 +28,17 @@ export const ProvideProvider = ({ children }: PropsWithChildren) => {
 
   const dispatch = useDispatch()
 
-  const [, avatarAddress] = parsePrefixedAddress(route.avatar)
+  const avatarAddress = parsePrefixedAddress(route.avatar)
   const avatarWaypoint = route.waypoints?.[route.waypoints.length - 1]
   const connectionType =
     avatarWaypoint &&
     'connection' in avatarWaypoint &&
     avatarWaypoint.connection.type
-  const [, connectedFrom] =
+  const connectedFrom =
     (avatarWaypoint &&
       'connection' in avatarWaypoint &&
       parsePrefixedAddress(avatarWaypoint.connection.from)) ||
-    []
+    ''
 
   const moduleAddress =
     connectionType === ConnectionType.IS_ENABLED ? connectedFrom : undefined

diff --git a/extension/src/panel/providers/isConnected.ts b/extension/src/panel/providers/isConnected.ts
@@ -18,7 +18,7 @@ export const isConnected = (
     return false
   }
 
-  const [, accountAddress] = parsePrefixedAddress(account)
+  const accountAddress = parsePrefixedAddress(account)
 
   return connection.accounts.some(
     (account) => account.toLowerCase() === accountAddress.toLowerCase(),