Refactor: show congrats notice if needed

Author: Momo Ozawa

WordPress/Classes/ViewRelated/Blog/Blog Dashboard/BlogDashboardViewController.swift

@@ -256,15 +256,16 @@ extension BlogDashboardViewController {
 
     private func showNoticeAsNeeded() {
         let quickStartGuide = QuickStartTourGuide.shared
+
         guard let tourToSuggest = quickStartGuide.tourToSuggest(for: blog) else {
+            quickStartGuide.showCongratsNoticeIfNeeded(for: blog)
             return
         }
 
         if quickStartGuide.tourInProgress {
             // If tour is in progress, show notice regardless of quickstart is shown in dashboard or my site
             quickStartGuide.suggest(tourToSuggest, for: blog)
-        }
-        else {
+        } else {
             guard shouldShowQuickStartChecklist() else {
                 return
             }

WordPress/Classes/ViewRelated/Blog/Blog Details/BlogDetailsViewController+FancyAlerts.swift

@@ -64,15 +64,16 @@ extension BlogDetailsViewController {
 
     private func showNoticeAsNeeded() {
         let quickStartGuide = QuickStartTourGuide.shared
+
         guard let tourToSuggest = quickStartGuide.tourToSuggest(for: blog) else {
+            quickStartGuide.showCongratsNoticeIfNeeded(for: blog)
             return
         }
 
         if quickStartGuide.tourInProgress {
             // If tour is in progress, show notice regardless of quickstart is shown in dashboard or my site
             quickStartGuide.suggest(tourToSuggest, for: blog)
-        }
-        else {
+        } else {
             guard shouldShowQuickStartChecklist() else {
                 return
             }

WordPress/Classes/ViewRelated/Blog/QuickStartTourGuide.swift

@@ -23,6 +23,9 @@ open class QuickStartTourGuide: NSObject {
     /// A flag indicating if the user is currently going through a tour or not.
     private(set) var tourInProgress = false
 
+    /// A flag indidcating whether we should show the congrats notice or not.
+    private var shouldShowCongratsNotice = false
+
     /// Represents the current entry point.
     @objc var currentEntryPoint: QuickStartTourEntryPoint = .unknown
 
@@ -229,6 +232,22 @@ open class QuickStartTourGuide: NSObject {
         completed(tour: tour, for: blog, postNotification: postNotification)
     }
 
+    func showCongratsNoticeIfNeeded(for blog: Blog) {
+        guard allToursCompleted(for: blog), shouldShowCongratsNotice else {
+            return
+        }
+
+        shouldShowCongratsNotice = false
+
+        let noticeStyle = QuickStartNoticeStyle(attributedMessage: nil, isDismissable: true)
+        let notice = Notice(title: Strings.congratulationsTitle,
+                            message: Strings.congratulationsMessage,
+                            style: noticeStyle,
+                            tag: noticeTag)
+
+        ActionDispatcher.dispatch(NoticeAction.post(notice))
+    }
+
     // we have this because poor stupid ObjC doesn't know what the heck an optional is
     @objc func currentElementInt() -> Int {
         return currentWaypoint()?.element.rawValue ?? NSNotFound
@@ -353,6 +372,7 @@ private extension QuickStartTourGuide {
             WPAnalytics.track(.quickStartAllToursCompleted)
             grantCongratulationsAward(for: blog)
             tourInProgress = false
+            shouldShowCongratsNotice = true
         } else {
             if let nextTour = tourToSuggest(for: blog) {
                 PushNotificationsManager.shared.postNotification(for: nextTour)
@@ -448,6 +468,11 @@ private extension QuickStartTourGuide {
         static let quickStartDelay: DispatchTimeInterval = .milliseconds(500)
         static let nextStepDelay: DispatchTimeInterval = .milliseconds(1000)
     }
+
+    private enum Strings {
+        static let congratulationsTitle = NSLocalizedString("Congrats! You know your way around", comment: "Title shown when all tours have been completed.")
+        static let congratulationsMessage = NSLocalizedString("Doesn't it feel good to cross things off a list?", comment: "Message shown when all tours have been completed")
+    }
 }
 
 internal extension NSNotification.Name {