Progress presenting the support screen from the migration flow

Author: salimbraksa

WordPress/Classes/ViewRelated/Support/SupportTableViewController.swift

@@ -171,7 +171,7 @@ private extension SupportTableViewController {
             )
         }
 
-        // Log out sections
+        // Log out Section
         var logOutSections: ImmuTableSection?
         if configuration.showsLogOutButton {
             let logOutRow = DestructiveButtonRow(

WordPress/Jetpack/Classes/System/JetpackWindowManager.swift

@@ -43,6 +43,6 @@ class JetpackWindowManager: WindowManager {
 
     // TODO: Add logic in here to trigger migration UI if needed
     private var shouldShowMigrationUI: Bool {
-        false
+        true
     }
 }

WordPress/Jetpack/Classes/ViewRelated/WordPress-to-Jetpack Migration/Common/MigrationDependencyContainer.swift

@@ -44,10 +44,14 @@ struct MigrationViewControllerFactory {
         }
     }
 
+    // MARK: - View Models
+
     private func makeWelcomeViewModel() -> MigrationWelcomeViewModel {
         MigrationWelcomeViewModel(account: makeAccount(), coordinator: coordinator)
     }
 
+    // MARK: - View Controllers
+
     private func makeWelcomeViewController() -> UIViewController {
         MigrationWelcomeViewController(viewModel: makeWelcomeViewModel())
     }

WordPress/Jetpack/Classes/ViewRelated/WordPress-to-Jetpack Migration/Common/Navigation/MigrationFlowCoordinator.swift

@@ -8,6 +8,9 @@ final class MigrationFlowCoordinator: ObservableObject {
     // related to this property.
     @Published private(set) var currentStep = MigrationStep.welcome
 
+    /// Call this closure to display the support screen
+    var routeToSupportViewController: (() -> Void)?
+
     func transitionToNextStep() {
         Task { [weak self] in
             if let nextStep = await Self.nextStep(from: currentStep) {

WordPress/Jetpack/Classes/ViewRelated/WordPress-to-Jetpack Migration/Common/Navigation/MigrationNavigationController.swift

@@ -2,13 +2,18 @@ import Combine
 import UIKit
 
 class MigrationNavigationController: UINavigationController {
+
+    // MARK: - Properties
+
     /// Navigation coordinator
     private let coordinator: MigrationFlowCoordinator
     /// The view controller factory used to push view controllers on the stack
     private let factory: MigrationViewControllerFactory
     /// Receives state changes to set the navigation stack accordingly
     private var cancellable: AnyCancellable?
 
+    // MARK: - Orientation
+
     override var supportedInterfaceOrientations: UIInterfaceOrientationMask {
         if WPDeviceIdentification.isiPhone() {
             return .portrait
@@ -21,6 +26,8 @@ class MigrationNavigationController: UINavigationController {
         .portrait
     }
 
+    // MARK: - Init
+
     init(coordinator: MigrationFlowCoordinator, factory: MigrationViewControllerFactory) {
         self.coordinator = coordinator
         self.factory = factory
@@ -36,6 +43,8 @@ class MigrationNavigationController: UINavigationController {
         fatalError("init(coder:) has not been implemented")
     }
 
+    // MARK: - Configuration
+
     private func configure() {
         let navigationBar = self.navigationBar
         let standardAppearance = UINavigationBarAppearance()
@@ -49,7 +58,15 @@ class MigrationNavigationController: UINavigationController {
             navigationBar.compactScrollEdgeAppearance = scrollEdgeAppearance
         }
         navigationBar.isTranslucent = true
-        listenForStateChanges()
+        configure(coordinator: coordinator)
+    }
+
+    private func configure(coordinator: MigrationFlowCoordinator) {
+        coordinator.routeToSupportViewController = { [weak self] in
+            let destination = SupportTableViewController(configuration: .currentAccountConfiguration())
+            self?.present(UINavigationController(rootViewController: destination), animated: true)
+        }
+        self.listenForStateChanges()
     }
 
     private func listenForStateChanges() {

WordPress/Jetpack/Classes/ViewRelated/WordPress-to-Jetpack Migration/Welcome/MigrationWelcomeViewModel.swift

@@ -24,12 +24,27 @@ final class MigrationWelcomeViewModel {
         let headerConfiguration = MigrationHeaderConfiguration(step: .welcome,
                                                                multiSite: blogListDataSource.visibleBlogsCount > 1)
 
-        let actionsConfiguration = MigrationActionsViewConfiguration(step: .welcome,
-                                                                     primaryHandler: { [weak coordinator] in
+        let primaryHandler = { [weak coordinator] () -> Void in
             coordinator?.transitionToNextStep()
-        },
-                                                                     secondaryHandler: { })
-
+        }
+        let secondaryHandler = { [weak coordinator] () -> Void in
+            // Which object is responsible for displaying the support screen?
+            //
+            // The following code needs to be executed somewhere:
+            //
+            // let destination = SupportTableViewController()
+            // presentingViewController.present(destination, completion: nil)
+            //
+            // Approaches considered:
+            //
+            // 1. The View Model shouldn't be responsible for displaying the support screen
+            // 2. The coordination can't perform the presentation because it doesn't have access to the `presenting` view controller
+            // 3. I thought of making the `MigrationViewControllerFactory` responsible for creating the `secondaryHandler` and injecting it into this view model
+            //    But that didn't work as well.
+            //
+            coordinator?.routeToSupportViewController?()
+        }
+        let actionsConfiguration = MigrationActionsViewConfiguration(step: .welcome, primaryHandler: primaryHandler, secondaryHandler: secondaryHandler)
         configuration = MigrationStepConfiguration(headerConfiguration: headerConfiguration, actionsConfiguration: actionsConfiguration)
     }
 }