Stop coordinator

johnnynanjiang · Oct 13, 2017 · 9fb41ee · 9fb41ee
1 parent 2cd4071
commit 9fb41ee
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 0 deletions.
diff --git a/Trust/AppCoordinator.swift b/Trust/AppCoordinator.swift
@@ -88,18 +88,21 @@ extension AppCoordinator: WelcomeViewControllerDelegate {
 extension AppCoordinator: TransactionCoordinatorDelegate {
     func didCancel(in coordinator: TransactionCoordinator) {
         coordinator.navigationController.dismiss(animated: true, completion: nil)
+        coordinator.stop()
         removeCoordinator(coordinator)
         reset()
     }
 
     func didChangeAccount(to account: Account, in coordinator: TransactionCoordinator) {
         coordinator.navigationController.dismiss(animated: true, completion: nil)
+        coordinator.stop()
         removeCoordinator(coordinator)
         showTransactions(for: account)
     }
 
     func didRestart(with account: Account, in coordinator: TransactionCoordinator) {
         coordinator.navigationController.dismiss(animated: true, completion: nil)
+        coordinator.stop()
         removeCoordinator(coordinator)
         showTransactions(for: account)
     }

diff --git a/Trust/Transactions/Coordinators/BalanceCoordinator.swift b/Trust/Transactions/Coordinators/BalanceCoordinator.swift
@@ -62,6 +62,10 @@ class BalanceCoordinator {
 
         delegate?.didUpdate(viewModel: viewModel)
     }
+
+    func stop() {
+
+    }
 }
 
 extension BalanceCoordinator: ExchangeRateCoordinatorDelegate {

diff --git a/Trust/Transactions/Coordinators/TransactionCoordinator.swift b/Trust/Transactions/Coordinators/TransactionCoordinator.swift
@@ -96,6 +96,11 @@ class TransactionCoordinator: Coordinator {
         navigationController.dismiss(animated: true, completion: nil)
     }
 
+    func stop() {
+        dataCoordinator.stop()
+        balanceCoordinator.stop()
+    }
+
     deinit {
         NotificationCenter.default.removeObserver(self)
     }
-Original file line number
+Diff line change
@@ Expand Up / @@ -62,6 +62,10 @@ class BalanceCoordinator { @@
             delegate?.didUpdate(viewModel: viewModel)
         }
+        func stop() {
+        }
     }
     extension BalanceCoordinator: ExchangeRateCoordinatorDelegate {
@@ Expand Down @@