Add reminders synced parameter to reminders sync success analytics event

Author: RenanLukas

WordPress/src/main/java/org/wordpress/android/bloggingreminders/BloggingRemindersSyncAnalyticsTracker.kt

@@ -10,14 +10,14 @@ class BloggingRemindersSyncAnalyticsTracker @Inject constructor(
 ) {
     fun trackStart() = analyticsTracker.track(Stat.BLOGGING_REMINDERS_SYNC_START)
 
-    fun trackSuccess() = analyticsTracker.track(Stat.BLOGGING_REMINDERS_SYNC_SUCCESS)
+    fun trackSuccess(remindersSyncedCount: Int) = analyticsTracker.track(
+            Stat.BLOGGING_REMINDERS_SYNC_SUCCESS, mapOf(REMINDERS_SYNCED_COUNT to remindersSyncedCount)
+    )
 
     fun trackFailed(errorType: ErrorType) =
             analyticsTracker.track(Stat.BLOGGING_REMINDERS_SYNC_FAILED, mapOf(ERROR_TYPE to errorType.value))
 
     sealed class ErrorType(val value: String) {
-        object NoBloggingRemindersFoundError : ErrorType("no_blogging_reminders_found_error")
-
         object QueryBloggingRemindersError : ErrorType("query_blogging_reminders_error")
 
         object UpdateBloggingRemindersError : ErrorType("update_blogging_reminders_error")
@@ -27,3 +27,5 @@ class BloggingRemindersSyncAnalyticsTracker @Inject constructor(
         }
     }
 }
+
+const val REMINDERS_SYNCED_COUNT = "reminders_synced_count"

WordPress/src/main/java/org/wordpress/android/bloggingreminders/resolver/BloggingRemindersResolver.kt

@@ -55,7 +55,7 @@ class BloggingRemindersResolver @Inject constructor(
                 val success = syncBloggingReminders(siteModelBloggingReminderMap)
                 if (success) onSuccess() else onFailure()
             } else {
-                bloggingRemindersSyncAnalyticsTracker.trackFailed(ErrorType.NoBloggingRemindersFoundError)
+                bloggingRemindersSyncAnalyticsTracker.trackSuccess(0)
                 onFailure()
             }
         } else {
@@ -85,11 +85,12 @@ class BloggingRemindersResolver @Inject constructor(
                     val isBloggingReminderAlreadySet = bloggingRemindersStore.bloggingRemindersModel(siteLocalId)
                             .first().enabledDays.isNotEmpty()
                     if (siteLocalId != 0 && !isBloggingReminderAlreadySet) {
+                        remindersSyncedCount = ++remindersSyncedCount
                         bloggingRemindersStore.updateBloggingReminders(bloggingReminder.copy(siteId = siteLocalId))
                     }
                 }
+                bloggingRemindersSyncAnalyticsTracker.trackSuccess(remindersSyncedCount)
             }
-            bloggingRemindersSyncAnalyticsTracker.trackSuccess()
             return true
         } catch (exception: Exception) {
             bloggingRemindersSyncAnalyticsTracker.trackFailed(ErrorType.UpdateBloggingRemindersError)