Fix #8721: Approve transaction does not show confirmation after editi…

…ng allowance (#8725)

* Fix case where `TransactionConfirmationStore` is torn down from observation when opening a detail view to edit approve tx permissions, which can cause spinner to show until transaction confirmation modal is dismissed because we miss the transaction updated observation method callback.

* Close stores on dismiss of modal to cover case where user manually dismisses modal RequestContainerView

Sources/BraveWallet/Panels/RequestContainerView.swift

@@ -28,9 +28,6 @@ struct RequestContainerView<DismissContent: ToolbarContent>: View {
               keyringStore: keyringStore,
               onDismiss: onDismiss
             )
-            .onDisappear {
-              cryptoStore.closeConfirmationStore()
-            }
           case .addSuggestedToken(let request):
             AddSuggestedTokenView(
               token: request.token,
@@ -62,9 +59,6 @@ struct RequestContainerView<DismissContent: ToolbarContent>: View {
               networkStore: cryptoStore.networkStore,
               onDismiss: onDismiss
             )
-            .onDisappear {
-              cryptoStore.closeSignMessageRequestStore()
-            }
           case let .signMessageError(signMessageErrors):
             SignMessageErrorView(
               signMessageErrors: signMessageErrors,
@@ -108,8 +102,11 @@ struct RequestContainerView<DismissContent: ToolbarContent>: View {
       }
     }
     .navigationViewStyle(.stack)
-    .onAppear {
-      // TODO: Fetch pending requests
+    .onDisappear {
+      // `onDisappear` on individual views will trigger for navigation pushes.
+      // Close stores when navigation covers manual dismiss & onDismiss() cases.
+      cryptoStore.closeConfirmationStore()
+      cryptoStore.closeSignMessageRequestStore()
     }
   }
 }