Send tasks to the full onboarding view

Author: itsmeichigo

WooCommerce/Classes/ViewRelated/Dashboard/DashboardView.swift

@@ -25,7 +25,7 @@ struct DashboardView: View {
     /// Set externally in the hosting controller.
     var onboardingTaskTapped: ((Site, StoreOnboardingTask) -> Void)?
     /// Set externally in the hosting controller.
-    var viewAllOnboardingTasksTapped: ((Site) -> Void)?
+    var viewAllOnboardingTasksTapped: ((Site, [StoreOnboardingTaskViewModel]) -> Void)?
 
     /// Set externally in the hosting controller.
     var showAllBlazeCampaignsTapped: (() -> Void)?
@@ -215,9 +215,9 @@ private extension DashboardView {
                                             onTaskTapped: { task in
                             guard let currentSite else { return }
                             onboardingTaskTapped?(currentSite, task)
-                        }, onViewAllTapped: {
+                        }, onViewAllTapped: { tasks in
                             guard let currentSite else { return }
-                            viewAllOnboardingTasksTapped?(currentSite)
+                            viewAllOnboardingTasksTapped?(currentSite, tasks)
                         })
                     case .blaze:
                         BlazeCampaignDashboardView(viewModel: viewModel.blazeCampaignDashboardViewModel,

WooCommerce/Classes/ViewRelated/Dashboard/DashboardViewHostingController.swift

@@ -175,11 +175,11 @@ private extension DashboardViewHostingController {
             storeOnboardingCoordinator?.start(task: task)
         }
 
-        rootView.viewAllOnboardingTasksTapped = { [weak self] site in
+        rootView.viewAllOnboardingTasksTapped = { [weak self] site, tasks in
             guard let self else { return }
             ServiceLocator.analytics.track(event: .DynamicDashboard.dashboardCardInteracted(type: .onboarding))
             updateStoreOnboardingCoordinatorIfNeeded(with: site)
-            storeOnboardingCoordinator?.start()
+            storeOnboardingCoordinator?.start(tasks: tasks)
         }
     }
 

WooCommerce/Classes/ViewRelated/Dashboard/Onboarding/StoreOnboardingCoordinator.swift

@@ -5,7 +5,7 @@ import struct Yosemite.Site
 import struct Yosemite.StoreOnboardingTask
 
 /// Coordinates navigation for store onboarding.
-final class StoreOnboardingCoordinator: Coordinator {
+final class StoreOnboardingCoordinator {
     typealias TaskType = StoreOnboardingTask.TaskType
 
     let navigationController: UINavigationController
@@ -31,11 +31,15 @@ final class StoreOnboardingCoordinator: Coordinator {
     }
 
     /// Navigates to the fullscreen store onboarding view.
-    func start() {
+    func start(tasks: [StoreOnboardingTaskViewModel]) {
         Task { @MainActor in
+            let viewModel = StoreOnboardingViewModel(
+                siteID: site.siteID,
+                isExpanded: true,
+                taskViewModels: tasks
+            )
             let onboardingNavigationController = UINavigationController()
-            let onboardingViewController = StoreOnboardingViewHostingController(viewModel: .init(siteID: site.siteID,
-                                                                                                 isExpanded: true),
+            let onboardingViewController = StoreOnboardingViewHostingController(viewModel: viewModel,
                                                                                 navigationController: onboardingNavigationController,
                                                                                 site: site)
             onboardingNavigationController.pushViewController(onboardingViewController, animated: false)

WooCommerce/Classes/ViewRelated/Dashboard/Onboarding/StoreOnboardingView.swift

@@ -42,9 +42,9 @@ final class StoreOnboardingViewHostingController: SelfSizingHostingController<St
         }
 
         if !viewModel.isExpanded {
-            rootView.viewAllTapped = { [weak self] in
+            rootView.viewAllTapped = { [weak self] tasks in
                 guard let self else { return }
-                self.coordinator.start()
+                self.coordinator.start(tasks: tasks)
             }
         }
     }
@@ -93,13 +93,13 @@ struct StoreOnboardingView: View {
     /// Set externally in the hosting controller.
     var taskTapped: ((StoreOnboardingTask) -> Void)?
     /// Set externally in the hosting controller.
-    var viewAllTapped: (() -> Void)?
+    var viewAllTapped: (([StoreOnboardingTaskViewModel]) -> Void)?
 
     @ObservedObject private var viewModel: StoreOnboardingViewModel
 
     init(viewModel: StoreOnboardingViewModel,
          onTaskTapped: ((StoreOnboardingTask) -> Void)? = nil,
-         onViewAllTapped: (() -> Void)? = nil) {
+         onViewAllTapped: (([StoreOnboardingTaskViewModel]) -> Void)? = nil) {
         self.viewModel = viewModel
         self.taskTapped = onTaskTapped
         self.viewAllTapped = onViewAllTapped
@@ -159,7 +159,9 @@ struct StoreOnboardingView: View {
                         .padding(.leading, Layout.padding)
 
                     // View all button
-                    viewAllButton(action: viewAllTapped, text: String(format: Localization.viewAll, viewModel.taskViewModels.count))
+                    viewAllButton(action: {
+                        viewAllTapped?(viewModel.taskViewModels)
+                    }, text: String(format: Localization.viewAll, viewModel.taskViewModels.count))
                         .renderedIf(viewModel.shouldShowViewAllButton)
                         .padding(.horizontal, Layout.padding)
                 }

WooCommerce/Classes/ViewRelated/Dashboard/Onboarding/StoreOnboardingViewModel.swift

@@ -83,6 +83,7 @@ class StoreOnboardingViewModel: ObservableObject {
     ///   - defaults: UserDefaults for storing when all onboarding tasks are completed
     init(siteID: Int64,
          isExpanded: Bool,
+         taskViewModels: [StoreOnboardingTaskViewModel] = [],
          stores: StoresManager = ServiceLocator.stores,
          defaults: UserDefaults = .standard,
          analytics: Analytics = ServiceLocator.analytics,
@@ -94,6 +95,7 @@ class StoreOnboardingViewModel: ObservableObject {
         self.defaults = defaults
         self.analytics = analytics
         self.waitingTimeTracker = waitingTimeTracker
+        self.taskViewModels = taskViewModels
 
         $noTasksAvailableForDisplay
             .combineLatest(defaults.publisher(for: \.completedAllStoreOnboardingTasks))