Track analytics about the number of total sites followed

wordpress-mobile · Feb 20, 2024 · 30d5acc · 30d5acc
1 parent 52a6334
commit 30d5acc
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderEvents.java b/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderEvents.java
@@ -51,6 +51,13 @@ public String getTagName() {
     }
 
     public static class FollowedBlogsChanged {
+        private final int mTotalSubscriptions;
+        public int getTotalSubscriptions() {
+            return mTotalSubscriptions;
+        }
+        public FollowedBlogsChanged(int totalSubscriptions) {
+            mTotalSubscriptions = totalSubscriptions;
+        }
     }
 
     public static class InterestTagsFetchEnded {

diff --git a/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.java b/WordPress/src/main/java/org/wordpress/android/ui/reader/ReaderPostListFragment.java
@@ -967,9 +967,7 @@ && hasCurrentTag()
             refreshPosts();
         }
 
-        ReaderBlogTable.getFollowedBlogs();
-        ReaderTracker.trackFollowedTagsCount(ReaderTagTable.getFollowedTags().size());
-        AppPrefs.setReaderAnalyticsCountTagsTimestamp(now);
+        ReaderTracker.trackFollowedSitesCount(event.getTotalSubscriptions());
     }
 
     @Override

diff --git a/...ress/src/main/java/org/wordpress/android/ui/reader/services/update/ReaderUpdateLogic.java b/...ress/src/main/java/org/wordpress/android/ui/reader/services/update/ReaderUpdateLogic.java
@@ -338,9 +338,10 @@ public void run() {
                     // changed if the server list doesn't have the same blogs as the local list
                     // (ie: a blog has been followed/unfollowed since local was last updated)
                     if (!localBlogs.hasSameBlogs(serverBlogs)) {
+                        final int totalSites = jsonObject == null ? 0 : jsonObject.optInt("total_subscriptions", 0);
                         ReaderPostTable.updateFollowedStatus();
                         AppLog.i(AppLog.T.READER, "reader blogs service > followed blogs changed");
-                        EventBus.getDefault().post(new ReaderEvents.FollowedBlogsChanged());
+                        EventBus.getDefault().post(new ReaderEvents.FollowedBlogsChanged(totalSites));
                     }
                 }