Fix ROLA completion slide up (#1040)

radixdlt · Mar 12, 2024 · ced4225 · ced4225
1 parent 091b86b
commit ced4225
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/RadixWallet/Features/DappInteractionFeature/Children/Completion/Completion.swift b/RadixWallet/Features/DappInteractionFeature/Children/Completion/Completion.swift
@@ -20,14 +20,14 @@ struct Completion: Sendable, FeatureReducer {
 		case dismissTapped
 	}
 
-	@Dependency(\.dismiss) var dismiss
+	enum DelegateAction: Sendable, Equatable {
+		case dismiss
+	}
 
 	func reduce(into state: inout State, viewAction: ViewAction) -> Effect<Action> {
 		switch viewAction {
 		case .dismissTapped:
-			.run { _ in
-				await dismiss()
-			}
+			.send(.delegate(.dismiss))
 		}
 	}
 }
diff --git a/RadixWallet/Features/DappInteractionFeature/Interactor/DappInteractor.swift b/RadixWallet/Features/DappInteractionFeature/Interactor/DappInteractor.swift
@@ -274,12 +274,9 @@ struct DappInteractor: Sendable, FeatureReducer {
 				return delayedMediumEffect(internal: .presentQueuedRequestIfNeeded)
 			}
 
-		case .modal(.dismiss):
-			if case .dappInteractionCompletion = state.currentModal {
-				return delayedMediumEffect(internal: .presentQueuedRequestIfNeeded)
-			}
-
-			return .none
+		case .modal(.presented(.dappInteractionCompletion(.delegate(.dismiss)))):
+			state.currentModal = nil
+			return delayedMediumEffect(internal: .presentQueuedRequestIfNeeded)
 
 		default:
 			return .none