Remove comments where we do not need migration

adborbas · adborbas · commit 5d7bfde976f9 · 2025-10-22T12:37:27.000+02:00
diff --git a/WooCommerce/Classes/ViewModels/Order Details/Refund Details/RefundDetailsViewModel.swift b/WooCommerce/Classes/ViewModels/Order Details/Refund Details/RefundDetailsViewModel.swift
@@ -128,7 +128,6 @@ extension RefundDetailsViewModel {
 
         case .orderItem:
             let item = refund.items[indexPath.row]
-            // TODO: Replace with ProductDetailRouter, native
             let loaderViewController = ProductLoaderViewController(model: .init(orderItemRefund: item),
                                                                    siteID: refund.siteID,
                                                                    forceReadOnly: true)
diff --git a/WooCommerce/Classes/ViewRelated/Dashboard/ProductStock/ProductStockDashboardCard.swift b/WooCommerce/Classes/ViewRelated/Dashboard/ProductStock/ProductStockDashboardCard.swift
@@ -189,7 +189,6 @@ private extension ProductStockDashboardCard {
                 .product(productID: item.productID)
             }
         }()
-        // TODO: Replace with ProductDetailRouter
         let loaderViewController = ProductLoaderViewController(model: model,
                                                                siteID: viewModel.siteID,
                                                                forceReadOnly: false)
diff --git a/WooCommerce/Classes/ViewRelated/Dashboard/TopPerformers/TopPerformerDataViewController.swift b/WooCommerce/Classes/ViewRelated/Dashboard/TopPerformers/TopPerformerDataViewController.swift
@@ -176,7 +176,6 @@ private extension TopPerformerDataViewController {
     /// Presents the product details for a given TopEarnerStatsItem.
     ///
     func presentProductDetails(statsItem: TopEarnerStatsItem) {
-        // TODO: Replace with ProductDetailRouter
         let loaderViewController = ProductLoaderViewController(model: .init(topEarnerStatsItem: statsItem),
                                                                siteID: siteID,
                                                                forceReadOnly: false)
diff --git a/WooCommerce/Classes/ViewRelated/Dashboard/TopPerformers/TopPerformersDashboardView.swift b/WooCommerce/Classes/ViewRelated/Dashboard/TopPerformers/TopPerformersDashboardView.swift
@@ -181,7 +181,6 @@ private extension TopPerformersDashboardView {
     }
 
     func productDetailView(for item: TopEarnerStatsItem) -> UIViewController {
-        // TODO: Replace with ProductDetailRouter
         let loaderViewController = ProductLoaderViewController(model: .init(topEarnerStatsItem: item),
                                                                siteID: viewModel.siteID,
                                                                forceReadOnly: false)
diff --git a/WooCommerce/Classes/ViewRelated/MainTabBarController.swift b/WooCommerce/Classes/ViewRelated/MainTabBarController.swift
@@ -1036,7 +1036,6 @@ private extension MainTabBarController {
                 return .productVariation(productID: productID, variationID: variationID)
             }
         }()
-        // TODO: Replace with ProductDetailRouter
         let productViewController = ProductLoaderViewController(model: model,
                                                                 siteID: error.siteID,
                                                                 forceReadOnly: false)
diff --git a/WooCommerce/Classes/ViewRelated/Orders/Order Details/Product List Section/Product Details/AggregatedProductListViewController.swift b/WooCommerce/Classes/ViewRelated/Orders/Order Details/Product List Section/Product Details/AggregatedProductListViewController.swift
@@ -122,7 +122,6 @@ private extension AggregatedProductListViewController {
     /// Displays the product details screen for the provided AggregateOrderItem
     ///
     func productWasPressed(orderItem: AggregateOrderItem) {
-        // TODO: Replace with ProductDetailRouter, native
         let loaderViewController = ProductLoaderViewController(model: .init(aggregateOrderItem: orderItem),
                                                                siteID: viewModel.order.siteID,
                                                                forceReadOnly: true)
diff --git a/WooCommerce/Classes/ViewRelated/Products/Product Loader/ProductLoaderView.swift b/WooCommerce/Classes/ViewRelated/Products/Product Loader/ProductLoaderView.swift
@@ -9,7 +9,6 @@ struct ProductLoaderView: UIViewControllerRepresentable {
     let forceReadOnly: Bool
 
     func makeUIViewController(context: Context) -> UINavigationController {
-        // TODO: Replace with ProductDetailRouter
         let viewController = ProductLoaderViewController(model: model,
                                                          siteID: siteID,
                                                          forceReadOnly: forceReadOnly)
diff --git a/WooCommerce/WooCommerceTests/ViewRelated/Products/ProductDetailsFactoryTests.swift b/WooCommerce/WooCommerceTests/ViewRelated/Products/ProductDetailsFactoryTests.swift
@@ -12,7 +12,6 @@ final class ProductDetailsFactoryTests: XCTestCase {
 
         let exp = expectation(description: #function)
         // Action
-        // TODO: Replace with ProductDetailRouter
         ProductDetailsFactory.productDetails(product: product,
                                              presentationStyle: .navigationStack,
                                              forceReadOnly: false) { viewController in
@@ -31,7 +30,6 @@ final class ProductDetailsFactoryTests: XCTestCase {
         let exp = expectation(description: #function)
 
         // Action
-        // TODO: Replace with ProductDetailRouter
         ProductDetailsFactory.productDetails(product: product,
                                              presentationStyle: .navigationStack,
                                              forceReadOnly: false) { viewController in
@@ -50,7 +48,6 @@ final class ProductDetailsFactoryTests: XCTestCase {
         let exp = expectation(description: #function)
 
         // Action
-        // TODO: Replace with ProductDetailRouter
         ProductDetailsFactory.productDetails(product: product,
                                              presentationStyle: .navigationStack,
                                              forceReadOnly: false) { viewController in
@@ -69,7 +66,6 @@ final class ProductDetailsFactoryTests: XCTestCase {
         let exp = expectation(description: #function)
 
         // Action
-        // TODO: Replace with ProductDetailRouter
         ProductDetailsFactory.productDetails(product: product,
                                              presentationStyle: .navigationStack,
                                              forceReadOnly: false) { viewController in
@@ -88,7 +84,6 @@ final class ProductDetailsFactoryTests: XCTestCase {
 
         // Action
         waitForExpectation { expectation in
-            // TODO: Replace with ProductDetailRouter
             ProductDetailsFactory.productDetails(product: product,
                                                  presentationStyle: .navigationStack,
                                                  forceReadOnly: false) { viewController in
@@ -105,7 +100,6 @@ final class ProductDetailsFactoryTests: XCTestCase {
 
         // Action
         waitForExpectation { expectation in
-            // TODO: Replace with ProductDetailRouter
             ProductDetailsFactory.productDetails(product: product,
                                                  presentationStyle: .navigationStack,
                                                  forceReadOnly: true) { viewController in

Original file line number	Diff line number	Diff line change
`@@ -189,7 +189,6 @@ private extension ProductStockDashboardCard {`
`189`	`189`	`.product(productID: item.productID)`
`190`	`190`	`}`
`191`	`191`	`}()`
`192`		`- // TODO: Replace with ProductDetailRouter`
`193`	`192`	`let loaderViewController = ProductLoaderViewController(model: model,`
`194`	`193`	`siteID: viewModel.siteID,`
`195`	`194`	`forceReadOnly: false)`
Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,6 @@ private extension TopPerformersDashboardView {`
`181`	`181`	`}`
`182`	`182`
`183`	`183`	`func productDetailView(for item: TopEarnerStatsItem) -> UIViewController {`
`184`		`- // TODO: Replace with ProductDetailRouter`
`185`	`184`	`let loaderViewController = ProductLoaderViewController(model: .init(topEarnerStatsItem: item),`
`186`	`185`	`siteID: viewModel.siteID,`
`187`	`186`	`forceReadOnly: false)`
Original file line number	Diff line number	Diff line change
`@@ -1036,7 +1036,6 @@ private extension MainTabBarController {`
`1036`	`1036`	`return .productVariation(productID: productID, variationID: variationID)`
`1037`	`1037`	`}`
`1038`	`1038`	`}()`
`1039`		`- // TODO: Replace with ProductDetailRouter`
`1040`	`1039`	`let productViewController = ProductLoaderViewController(model: model,`
`1041`	`1040`	`siteID: error.siteID,`
`1042`	`1041`	`forceReadOnly: false)`