Show bookings tab when site is eligible

itsmeichigo · itsmeichigo · commit 077d33f5a7b2 · 2025-09-19T13:17:12.000+07:00
diff --git a/WooCommerce/Classes/Bookings/BookingsTabView.swift b/WooCommerce/Classes/Bookings/BookingsTabView.swift
@@ -0,0 +1,39 @@
+import SwiftUI
+
+/// Hosting view for `BookingsTabView`
+///
+final class BookingsTabViewHostingController: UIHostingController<BookingsTabView> {
+    init(siteID: Int64) {
+        super.init(rootView: BookingsTabView())
+        configureTabBarItem()
+    }
+
+    @MainActor @preconcurrency required dynamic init?(coder aDecoder: NSCoder) {
+        fatalError("init(coder:) has not been implemented")
+    }
+
+    override var shouldShowOfflineBanner: Bool {
+        return true
+    }
+}
+
+private extension BookingsTabViewHostingController {
+    func configureTabBarItem() {
+        tabBarItem.image = UIImage(systemName: "calendar")
+        tabBarItem.title = "Bookings"
+        tabBarItem.accessibilityIdentifier = "tab-bar-bookings-item"
+    }
+}
+
+/// Main content of the Bookings tab
+///
+struct BookingsTabView: View {
+    var body: some View {
+        NavigationSplitView {
+            Text("Booking List")
+        } detail: {
+            Text("Booking Detail Screen")
+        }
+
+    }
+}
diff --git a/WooCommerce/Classes/ViewRelated/MainTabBarController.swift b/WooCommerce/Classes/ViewRelated/MainTabBarController.swift
@@ -23,6 +23,10 @@ enum WooTab {
     ///
     case products
 
+    /// Bookings Tab
+    ///
+    case bookings
+
     /// Point of Sale Tab
     ///
     case pointOfSale
@@ -38,15 +42,18 @@ extension WooTab {
     /// - Parameters:
     ///   - visibleIndex: the index of visible tabs on the tab bar
     ///   - isPOSTabVisible: indicates if the Point of Sale tab is visible.
-    init(visibleIndex: Int, isPOSTabVisible: Bool) {
-        let tabs = WooTab.visibleTabs(isPOSTabVisible: isPOSTabVisible)
+    ///   - isBookingsTabVisible: indicates if the Bookings tab is visible.
+    init(visibleIndex: Int, isPOSTabVisible: Bool, isBookingsTabVisible: Bool = false) {
+        let tabs = WooTab.visibleTabs(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
         self = tabs[visibleIndex]
     }
 
     /// Returns the visible tab index.
-    /// - Parameter isPOSTabVisible: indicates if the Point of Sale tab is visible.
-    func visibleIndex(isPOSTabVisible: Bool) -> Int {
-        let tabs = WooTab.visibleTabs(isPOSTabVisible: isPOSTabVisible)
+    /// - Parameters:
+    ///   - isPOSTabVisible: indicates if the Point of Sale tab is visible.
+    ///   - isBookingsTabVisible: indicates if the Bookings tab is visible.
+    func visibleIndex(isPOSTabVisible: Bool, isBookingsTabVisible: Bool = false) -> Int {
+        let tabs = WooTab.visibleTabs(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
         guard let tabIndex = tabs.firstIndex(where: { $0 == self }) else {
             assertionFailure("Trying to get the visible tab index for tab \(self) while the visible tabs are: \(tabs)")
             return 0
@@ -56,14 +63,23 @@ extension WooTab {
 
     /// Note: currently only the Dashboard tab (My Store) view controller is set up in Main.storyboard.
     ///
-    /// - Parameter isPOSTabVisible: indicates if the Point of Sale tab is visible.
+    /// - Parameters:
+    ///   - isPOSTabVisible: indicates if the Point of Sale tab is visible.
+    ///   - isBookingsTabVisible: indicates if the Bookings tab is visible.
     /// - Returns: visible tabs in the tab bar.
-    static func visibleTabs(isPOSTabVisible: Bool) -> [WooTab] {
+    static func visibleTabs(isPOSTabVisible: Bool, isBookingsTabVisible: Bool = false) -> [WooTab] {
+        var tabs: [WooTab] = [.myStore, .orders, .products]
+
+        if isBookingsTabVisible {
+            tabs.append(.bookings)
+        }
+
         if isPOSTabVisible {
-            return [.myStore, .orders, .products, .pointOfSale, .hubMenu]
-        } else {
-            return [.myStore, .orders, .products, .hubMenu]
+            tabs.append(.pointOfSale)
         }
+
+        tabs.append(.hubMenu)
+        return tabs
     }
 }
 
@@ -114,6 +130,8 @@ final class MainTabBarController: UITabBarController {
     private let posContainerController = TabContainerController()
     private var posTabCoordinator: POSTabCoordinator?
 
+    private let bookingsContainerController = TabContainerController()
+
     private let hubMenuContainerController = TabContainerController()
     private var hubMenuTabCoordinator: HubMenuCoordinator?
 
@@ -132,7 +150,11 @@ final class MainTabBarController: UITabBarController {
     private var posEligibilityCheckTask: Task<Void, Never>?
     private var posCatalogSyncCoordinator: POSCatalogSyncCoordinatorProtocol?
 
+    private var bookingsEligibilityChecker: BookingsTabEligibilityCheckerProtocol?
+    private var bookingsEligibilityCheckTask: Task<Void, Never>?
+
     private var isPOSTabVisible: Bool = false
+    private var isBookingsTabVisible: Bool = false
 
     private lazy var isProductsSplitViewFeatureFlagOn = featureFlagService.isFeatureFlagEnabled(.splitViewInProductsTab)
 
@@ -182,6 +204,7 @@ final class MainTabBarController: UITabBarController {
     deinit {
         cancellableSiteID?.cancel()
         posEligibilityCheckTask?.cancel()
+        bookingsEligibilityCheckTask?.cancel()
     }
 
     // MARK: - Overridden Methods
@@ -192,8 +215,8 @@ final class MainTabBarController: UITabBarController {
 
         delegate = self
 
-        // POS tab is hidden by default.
-        updateTabViewControllers(isPOSTabVisible: false)
+        // POS and Bookings tabs are hidden by default.
+        updateTabViewControllers(isPOSTabVisible: false, isBookingsTabVisible: false)
         observeSiteIDForViewControllers()
         observeProductImageUploadStatusUpdates()
 
@@ -220,11 +243,11 @@ final class MainTabBarController: UITabBarController {
     }
 
     override func tabBar(_ tabBar: UITabBar, didSelect item: UITabBarItem) {
-        let currentlySelectedTab = WooTab(visibleIndex: selectedIndex, isPOSTabVisible: isPOSTabVisible)
+        let currentlySelectedTab = WooTab(visibleIndex: selectedIndex, isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
         guard let userSelectedIndex = tabBar.items?.firstIndex(of: item) else {
                 return
         }
-        let userSelectedTab = WooTab(visibleIndex: userSelectedIndex, isPOSTabVisible: isPOSTabVisible)
+        let userSelectedTab = WooTab(visibleIndex: userSelectedIndex, isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
 
         // Did we reselect the already-selected tab?
         if currentlySelectedTab == userSelectedTab {
@@ -254,7 +277,7 @@ final class MainTabBarController: UITabBarController {
     func navigateToTabWithViewController(_ tab: WooTab, animated: Bool = false, completion: ((UIViewController) -> Void)? = nil) {
         dismiss(animated: animated) { [weak self] in
             guard let self else { return }
-            selectedIndex = tab.visibleIndex(isPOSTabVisible: isPOSTabVisible)
+            selectedIndex = tab.visibleIndex(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
             guard let selectedViewController else {
                 return
             }
@@ -361,6 +384,9 @@ private extension MainTabBarController {
         case .products:
             ServiceLocator.analytics.track(
                 event: .Products.productListSelected(horizontalSizeClass: UITraitCollection.current.horizontalSizeClass))
+        case .bookings:
+            // TODO: Add bookings tab selected analytics
+            break
         case .hubMenu:
             ServiceLocator.analytics.track(.hubMenuTabSelected)
         case .pointOfSale:
@@ -381,6 +407,9 @@ private extension MainTabBarController {
         case .products:
             ServiceLocator.analytics.track(
                 event: .Products.productListReselected(horizontalSizeClass: UITraitCollection.current.horizontalSizeClass))
+        case .bookings:
+            // TODO: Add bookings tab reselected analytics
+            break
         case .hubMenu:
             ServiceLocator.analytics.track(.hubMenuTabReselected)
             break
@@ -667,14 +696,14 @@ extension MainTabBarController: DeepLinkNavigator {
 private extension MainTabBarController {
     func observePOSEligibilityForPOSTabVisibility(siteID: Int64) {
         guard let posEligibilityChecker else {
-            updateTabViewControllers(isPOSTabVisible: false)
+            updateTabViewControllers(isPOSTabVisible: false, isBookingsTabVisible: isBookingsTabVisible)
             viewModel.loadHubMenuTabBadge()
             return
         }
 
         // Sets POS tab initial visibility based on cached value if available.
         let initialVisibility = posEligibilityChecker.checkInitialVisibility()
-        updateTabViewControllers(isPOSTabVisible: initialVisibility)
+        updateTabViewControllers(isPOSTabVisible: initialVisibility, isBookingsTabVisible: isBookingsTabVisible)
 
         // Cancels any existing task.
         posEligibilityCheckTask?.cancel()
@@ -685,7 +714,7 @@ private extension MainTabBarController {
             let isPOSTabVisible = await posEligibilityChecker.checkVisibility()
             analytics.track(.pointOfSaleTabVisibilityChecked, withProperties: ["is_visible": isPOSTabVisible])
             cachePOSTabVisibility(siteID: siteID, isPOSTabVisible: isPOSTabVisible)
-            updateTabViewControllers(isPOSTabVisible: isPOSTabVisible)
+            updateTabViewControllers(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
             viewModel.loadHubMenuTabBadge()
 
             // Trigger POS catalog sync if tab is visible and feature flag is enabled
@@ -695,19 +724,20 @@ private extension MainTabBarController {
         }
     }
 
-    func updateTabViewControllers(isPOSTabVisible: Bool) {
-        guard isPOSTabVisible != self.isPOSTabVisible || (viewControllers?.count ?? 0) == 0 else {
+    func updateTabViewControllers(isPOSTabVisible: Bool, isBookingsTabVisible: Bool = false) {
+        guard isPOSTabVisible != self.isPOSTabVisible || isBookingsTabVisible != self.isBookingsTabVisible || (viewControllers?.count ?? 0) == 0 else {
             return
         }
         var controllers = [UIViewController]()
-        let tabs = WooTab.visibleTabs(isPOSTabVisible: isPOSTabVisible)
+        let tabs = WooTab.visibleTabs(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
         tabs.forEach { tab in
-            let tabIndex = tab.visibleIndex(isPOSTabVisible: isPOSTabVisible)
+            let tabIndex = tab.visibleIndex(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
             let tabViewController = rootTabViewController(tab: tab)
             controllers.insert(tabViewController, at: tabIndex)
         }
         viewControllers = controllers
         self.isPOSTabVisible = isPOSTabVisible
+        self.isBookingsTabVisible = isBookingsTabVisible
     }
 
     func rootTabViewController(tab: WooTab) -> UIViewController {
@@ -718,6 +748,8 @@ private extension MainTabBarController {
                 return ordersContainerController
             case .products:
                 return isProductsSplitViewFeatureFlagOn ? productsContainerController: productsNavigationController
+            case .bookings:
+                return bookingsContainerController
             case .hubMenu:
                 return hubMenuContainerController
             case .pointOfSale:
@@ -757,6 +789,12 @@ private extension MainTabBarController {
                                                                                    navigateToContent: { _ in })]
         }
 
+        // Configures Booking tab.
+        let bookingsEligibilityChecker = BookingsTabEligibilityChecker(site: site)
+        self.bookingsEligibilityChecker = bookingsEligibilityChecker
+        let bookingsViewController = createBookingsViewController(siteID: site.siteID)
+        bookingsContainerController.wrappedController = bookingsViewController
+
         // Configures POS tab coordinator once per logged in site session.
         let posEligibilityChecker = posEligibilityCheckerFactory(site)
         self.posEligibilityChecker = posEligibilityChecker
@@ -780,9 +818,10 @@ private extension MainTabBarController {
         hubMenuTabCoordinator?.activate(siteID: siteID)
 
         // Set dashboard to be the default tab.
-        selectedIndex = WooTab.myStore.visibleIndex(isPOSTabVisible: isPOSTabVisible)
+        selectedIndex = WooTab.myStore.visibleIndex(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
 
         observePOSEligibilityForPOSTabVisibility(siteID: siteID)
+        observeBookingsEligibilityForBookingsTabVisibility(site: site)
     }
 
     func createDashboardViewController(siteID: Int64) -> UIViewController {
@@ -816,6 +855,10 @@ private extension MainTabBarController {
         }
     }
 
+    func createBookingsViewController(siteID: Int64) -> UIViewController {
+        BookingsTabViewHostingController(siteID: siteID)
+    }
+
     func createHubMenuTabCoordinator() -> HubMenuCoordinator {
         HubMenuCoordinator(tabContainerController: hubMenuContainerController,
                            storesManager: stores,
@@ -828,6 +871,29 @@ private extension MainTabBarController {
             }
         })
     }
+
+    func observeBookingsEligibilityForBookingsTabVisibility(site: Site) {
+        guard let bookingsEligibilityChecker else {
+            updateTabViewControllers(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: false)
+            return
+        }
+
+        // Sets Bookings tab initial visibility based on cached value if available.
+        let initialVisibility = bookingsEligibilityChecker.checkInitialVisibility()
+        updateTabViewControllers(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: initialVisibility)
+
+        // Cancels any existing task.
+        bookingsEligibilityCheckTask?.cancel()
+
+        // Starts observing the Bookings eligibility state.
+        bookingsEligibilityCheckTask = Task { @MainActor [weak self] in
+            guard let self, let bookingsEligibilityChecker = self.bookingsEligibilityChecker else { return }
+            let isBookingsTabVisible = await bookingsEligibilityChecker.checkVisibility()
+            // TODO: Add analytics tracking for bookings tab visibility
+            cacheBookingsTabVisibility(siteID: site.siteID, isBookingsTabVisible: isBookingsTabVisible)
+            updateTabViewControllers(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
+        }
+    }
 }
 
 // MARK: - Hub Menu Tab Badge Updates
@@ -848,7 +914,7 @@ private extension MainTabBarController {
 
     func updateMenuTabBadge(with action: NotificationBadgeActionType) {
         let tab = WooTab.hubMenu
-        let tabIndex = tab.visibleIndex(isPOSTabVisible: isPOSTabVisible)
+        let tabIndex = tab.visibleIndex(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
         let input = NotificationsBadgeInput(action: action, tab: tab, tabBar: self.tabBar, tabIndex: tabIndex)
 
         self.notificationsBadge.updateBadge(with: input)
@@ -865,7 +931,7 @@ private extension MainTabBarController {
             }
 
             let tab = WooTab.orders
-            let tabIndex = tab.visibleIndex(isPOSTabVisible: isPOSTabVisible)
+            let tabIndex = tab.visibleIndex(isPOSTabVisible: isPOSTabVisible, isBookingsTabVisible: isBookingsTabVisible)
 
             guard let orderTab: UITabBarItem = self.tabBar.items?[tabIndex] else {
                 return
@@ -983,6 +1049,10 @@ private extension MainTabBarController {
     func cachePOSTabVisibility(siteID: Int64, isPOSTabVisible: Bool) {
         posEligibilityService.cachePOSTabVisibility(siteID: siteID, isVisible: isPOSTabVisible)
     }
+
+    func cacheBookingsTabVisibility(siteID: Int64, isBookingsTabVisible: Bool) {
+        UserDefaults.standard.cacheBookingsTabVisibility(siteID: siteID, isVisible: isBookingsTabVisible)
+    }
 }
 
 private extension MainTabBarController {
diff --git a/WooCommerce/Classes/ViewRelated/TabBar/WooTab+Tag.swift b/WooCommerce/Classes/ViewRelated/TabBar/WooTab+Tag.swift
@@ -10,10 +10,12 @@ extension WooTab {
             return 1
         case .products:
             return 2
-        case .pointOfSale:
+        case .bookings:
             return 3
-        case .hubMenu:
+        case .pointOfSale:
             return 4
+        case .hubMenu:
+            return 5
         }
     }
 }
diff --git a/WooCommerce/WooCommerce.xcodeproj/project.pbxproj b/WooCommerce/WooCommerce.xcodeproj/project.pbxproj
@@ -1646,7 +1646,6 @@
 		68A38DF52B293B030090C263 /* MockProductListViewModel.swift in Sources */ = {isa = PBXBuildFile; fileRef = 68A38DF42B293B030090C263 /* MockProductListViewModel.swift */; };
 		68A5221B2BA1804900A6A584 /* PluginDetailsViewModelTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 68A5221A2BA1804900A6A584 /* PluginDetailsViewModelTests.swift */; };
 		68A905012ACCFC13004C71D3 /* CollapsibleProductCard.swift in Sources */ = {isa = PBXBuildFile; fileRef = 68A905002ACCFC13004C71D3 /* CollapsibleProductCard.swift */; };
-		68AC9D292ACE598B0042F784 /* ProductImageThumbnail.swift in Sources */ = {isa = PBXBuildFile; fileRef = 68AC9D282ACE598B0042F784 /* ProductImageThumbnail.swift */; };
 		68B3BA262D9147480000B2F2 /* AISettingsView.swift in Sources */ = {isa = PBXBuildFile; fileRef = 68B3BA252D9147440000B2F2 /* AISettingsView.swift */; };
 		68B681162D9257810098D5CD /* PointOfSaleCouponsController.swift in Sources */ = {isa = PBXBuildFile; fileRef = 68B681152D92577F0098D5CD /* PointOfSaleCouponsController.swift */; };
 		68B6F22B2ADE7ED500D171FC /* TooltipView.swift in Sources */ = {isa = PBXBuildFile; fileRef = 68B6F22A2ADE7ED500D171FC /* TooltipView.swift */; };
@@ -4846,7 +4845,6 @@
 		68A38DF42B293B030090C263 /* MockProductListViewModel.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = MockProductListViewModel.swift; sourceTree = "<group>"; };
 		68A5221A2BA1804900A6A584 /* PluginDetailsViewModelTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = PluginDetailsViewModelTests.swift; sourceTree = "<group>"; };
 		68A905002ACCFC13004C71D3 /* CollapsibleProductCard.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = CollapsibleProductCard.swift; sourceTree = "<group>"; };
-		68AC9D282ACE598B0042F784 /* ProductImageThumbnail.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ProductImageThumbnail.swift; sourceTree = "<group>"; };
 		68B3BA252D9147440000B2F2 /* AISettingsView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = AISettingsView.swift; sourceTree = "<group>"; };
 		68B681152D92577F0098D5CD /* PointOfSaleCouponsController.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = PointOfSaleCouponsController.swift; sourceTree = "<group>"; };
 		68B6F22A2ADE7ED500D171FC /* TooltipView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = TooltipView.swift; sourceTree = "<group>"; };

Original file line number	Diff line number	Diff line change
`@@ -10,10 +10,12 @@ extension WooTab {`
`10`	`10`	`return 1`
`11`	`11`	`case .products:`
`12`	`12`	`return 2`
`13`		`- case .pointOfSale:`
	`13`	`+ case .bookings:`
`14`	`14`	`return 3`
`15`		`- case .hubMenu:`
	`15`	`+ case .pointOfSale:`
`16`	`16`	`return 4`
	`17`	`+ case .hubMenu:`
	`18`	`+ return 5`
`17`	`19`	`}`
`18`	`20`	`}`
`19`	`21`	`}`