Fix: prompt -> confirm (#3939)

* Fix: prompt -> confirm * Update unit test
safe-global · Jul 12, 2024 · cdec625 · cdec625
1 parent 25b58bf
commit cdec625
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/services/safe-wallet-provider/useSafeWalletProvider.test.tsx b/src/services/safe-wallet-provider/useSafeWalletProvider.test.tsx
@@ -300,7 +300,7 @@ describe('useSafeWalletProvider', () => {
       } as unknown as router.NextRouter)
 
       // @ts-expect-error - auto accept prompt
-      jest.spyOn(window, 'prompt').mockReturnValue(true)
+      jest.spyOn(window, 'confirm').mockReturnValue(true)
 
       const { result } = renderHook(() => _useTxFlowApi('1', '0x1234567890000000000000000000000000000000'), {
         initialReduxState: {

diff --git a/src/services/safe-wallet-provider/useSafeWalletProvider.tsx b/src/services/safe-wallet-provider/useSafeWalletProvider.tsx
@@ -170,7 +170,7 @@ export const _useTxFlowApi = (chainId: string, safeAddress: string): WalletSDK |
           throw new Error(`Chain ${chainId} not supported`)
         }
 
-        if (prompt(`${appInfo.name} wants to switch to ${cfg.shortName}. Do you want to continue?`)) {
+        if (confirm(`${appInfo.name} wants to switch to ${cfg.shortName}. Do you want to continue?`)) {
           router.push({
             pathname: AppRoutes.index,
             query: {