Contain dismissal logic to view model only

Author: iamgabrielma

WooCommerce/Classes/ViewRelated/Orders/OrderListViewController.swift

@@ -797,7 +797,7 @@ private extension OrderListViewController {
         let shareIPPFeedbackAction = TopBannerViewModel.ActionButton(title: Localization.shareFeedbackButton, action: { [weak self] _ in
             self?.displayIPPFeedbackBannerSurvey()
             // We dismiss the banner at this point as we cannot know if the user successfully submitted it
-            self?.viewModel.dismissIPPFeedbackBanner(remindAfterDays: nil)
+            self?.viewModel.IPPFeedbackWasDismissed()
         })
 
         let viewModel = TopBannerViewModel(

WooCommerce/Classes/ViewRelated/Orders/OrderListViewModel.swift

@@ -377,6 +377,10 @@ extension OrderListViewModel {
     func dontShowAgainTapped() {
         dismissIPPFeedbackBanner(remindAfterDays: nil)
     }
+
+    func IPPFeedbackWasDismissed() {
+        dismissIPPFeedbackBanner(remindAfterDays: nil)
+    }
 }
 
 // MARK: - Remote Notifications Observation