[Tracks] Send site related properties in application level tracking events (#15173)

Author: selanthiraiyan

WooCommerce/Classes/Analytics/WooAnalyticsStat.swift

@@ -1316,14 +1316,11 @@ extension WooAnalyticsStat {
     /// Indicates if site information should be included with this event when it's sent to the tracks server.
     /// Returns `true` if it should be included, `false` otherwise.
     ///
-    /// Note: Currently all application-level and authentication events will return false. If you wish
+    /// Note: Currently all authentication events will return false. If you wish
     /// to include additional no-site-info events, please add them here.
     ///
     var shouldSendSiteProperties: Bool {
         switch self {
-        // Application events
-        case .applicationClosed, .applicationOpened, .applicationUpgraded, .applicationInstalled, .watchAppOpened:
-            return false
         // Authentication Events
         case .signedIn, .logout, .openedLogin, .loginFailed,
              .loginAutoFillCredentialsFilled, .loginAutoFillCredentialsUpdated, .loginEmailFormViewed, .loginMagicLinkOpenEmailClientViewed,

WooCommerce/Classes/Yosemite/DefaultStoresManager.swift

@@ -534,6 +534,26 @@ private extension DefaultStoresManager {
         dispatch(action)
     }
 
+    ///  Implements a retry logic for fetching the store information i.e. `store_id` to reduce the chances of it being missing.
+    ///  We use three attempts with an exponential backoff.
+    ///
+    @MainActor
+    func fetchSystemInformationAndRetryIfFails(siteID: Int64,
+                                               retryCount: Int = 0) async -> SystemInformation? {
+        guard retryCount <= 3 else {
+            return nil
+        }
+
+        let waitTime = Int(pow(Double(2), Double(retryCount)))
+        try? await Task.sleep(for: .seconds(waitTime))
+
+        if let info = await synchronizeSystemInformation(siteID: siteID) {
+            return info
+        } else {
+            return await fetchSystemInformationAndRetryIfFails(siteID: siteID, retryCount: retryCount + 1)
+        }
+    }
+
     /// Synchronizes all system information for the store with specified ID.
     /// When finished, loads the store uuid into the session.
     ///
@@ -646,7 +666,7 @@ private extension DefaultStoresManager {
         Task { @MainActor in
             // Order statuses and system plugins syncing are required outside of snapshot tracking.
             async let orderStatuses = retrieveOrderStatus(with: siteID)
-            async let systemInformation = synchronizeSystemInformation(siteID: siteID)
+            async let systemInformation = fetchSystemInformationAndRetryIfFails(siteID: siteID)
 
             trackSnapshotIfNeeded(siteID: siteID, orderStatuses: await orderStatuses, systemPlugins: await systemInformation?.systemPlugins)
         }