also clear transactions when editing the current connection's avatar

gnosisguild · Mar 4, 2024 · 9ecebc3 · 9ecebc3
1 parent 867701b
commit 9ecebc3
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 9 deletions.
diff --git a/extension/src/connections/ConnectionsDrawer/Edit/index.tsx b/extension/src/connections/ConnectionsDrawer/Edit/index.tsx
@@ -86,7 +86,7 @@ const EditConnection: React.FC<Props> = ({ connectionId, onLaunched }) => {
     }
 
     const confirmation = await getConfirmation(
-      'Switching the piloted Safe will empty your current transaction bundle.'
+      'Switching the Piloted Safe will empty your current transaction bundle.'
     )
 
     if (!confirmation) {
@@ -217,13 +217,20 @@ const EditConnection: React.FC<Props> = ({ connectionId, onLaunched }) => {
               <AvatarInput
                 availableSafes={safes}
                 value={avatarAddress}
-                onChange={(address) =>
-                  updateConnection({
-                    avatarAddress: address,
-                    moduleAddress: '',
-                    moduleType: undefined,
-                  })
-                }
+                onChange={async (address) => {
+                  const keepTransactionBundle =
+                    address.toLowerCase() ===
+                    connection.avatarAddress.toLowerCase()
+                  const confirmed =
+                    keepTransactionBundle || (await confirmClearTransactions())
+                  if (confirmed) {
+                    updateConnection({
+                      avatarAddress: address,
+                      moduleAddress: '',
+                      moduleType: undefined,
+                    })
+                  }
+                }}
               />
             </Field>
             <Field label="Zodiac Mod" disabled={modules.length === 0}>

diff --git a/extension/src/connections/ConnectionsDrawer/List/index.tsx b/extension/src/connections/ConnectionsDrawer/List/index.tsx
@@ -48,7 +48,7 @@ const ConnectionItem: React.FC<ConnectionItemProps> = ({
     }
 
     const confirmation = await getConfirmation(
-      'Switching the piloted Safe will empty your current transaction bundle.'
+      'Switching the Piloted Safe will empty your current transaction bundle.'
     )
 
     if (!confirmation) {