Rename dismiss methods to be more explicit

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.IPPFeedbackWasDismissed()
+            self?.viewModel.IPPFeedbackBannerWasDismissed()
         })
 
         let viewModel = TopBannerViewModel(
@@ -829,12 +829,12 @@ private extension OrderListViewController {
         )
 
         let remindMeLaterAction = UIAlertAction( title: Localization.remindMeLater, style: .default) { [weak self] _ in
-            self?.viewModel.remindMeLaterTapped()
+            self?.viewModel.IPPFeedbackBannerRemindMeLaterTapped()
         }
         actionSheet.addAction(remindMeLaterAction)
 
         let dontShowAgainAction = UIAlertAction( title: Localization.dontShowAgain, style: .default) { [weak self] _ in
-            self?.viewModel.dontShowAgainTapped()
+            self?.viewModel.IPPFeedbackBannerDontShowAgainTapped()
         }
         actionSheet.addAction(dontShowAgainAction)
 

WooCommerce/Classes/ViewRelated/Orders/OrderListViewModel.swift

@@ -370,15 +370,15 @@ extension OrderListViewModel {
         stores.dispatch(updateBannerVisibility)
     }
 
-    func remindMeLaterTapped() {
+    func IPPFeedbackBannerRemindMeLaterTapped() {
         dismissIPPFeedbackBanner(remindAfterDays: Constants.remindIPPBannerDismissalAfterDays)
     }
 
-    func dontShowAgainTapped() {
+    func IPPFeedbackBannerDontShowAgainTapped() {
         dismissIPPFeedbackBanner(remindAfterDays: nil)
     }
 
-    func IPPFeedbackWasDismissed() {
+    func IPPFeedbackBannerWasDismissed() {
         dismissIPPFeedbackBanner(remindAfterDays: nil)
     }
 }