chore: remove redundant popTransactionStack call on approve tx view (#4236)

Author: enesozturk

apps/laboratory/tests/shared/validators/ModalValidator.ts

@@ -64,7 +64,7 @@ export class ModalValidator {
     await expect(
       this.page.getByTestId('w3m-authentication-status'),
       'Authentication status should be: unauthenticated'
-    ).toContainText('unauthenticated', { timeout: 15000 })
+    ).toContainText('unauthenticated', { timeout: 20000 })
   }
 
   async expectOnSignOutEventCalled(toBe: boolean) {

packages/scaffold-ui/src/views/w3m-approve-transaction-view/index.ts

@@ -2,12 +2,7 @@ import { LitElement, html } from 'lit'
 import { state } from 'lit/decorators.js'
 
 import { getW3mThemeVariables } from '@reown/appkit-common'
-import {
-  ConnectorController,
-  ModalController,
-  RouterController,
-  ThemeController
-} from '@reown/appkit-controllers'
+import { ConnectorController, ModalController, ThemeController } from '@reown/appkit-controllers'
 import { customElement } from '@reown/appkit-ui'
 
 import styles from './styles.js'
@@ -38,7 +33,6 @@ export class W3mApproveTransactionView extends LitElement {
         ModalController.subscribeKey('open', isOpen => {
           if (!isOpen) {
             this.onHideIframe()
-            RouterController.popTransactionStack()
           }
         }),
         ModalController.subscribeKey('shake', val => {