Skip to content

Commit 2267492

Browse files
Include site data with all events except those related to authentication, this will make sure both the blog_id and store_id properties are set when they are present.
1 parent 7b16022 commit 2267492

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

WooCommerce/Classes/Analytics/WooAnalyticsStat.swift

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1311,14 +1311,11 @@ extension WooAnalyticsStat {
13111311
/// Indicates if site information should be included with this event when it's sent to the tracks server.
13121312
/// Returns `true` if it should be included, `false` otherwise.
13131313
///
1314-
/// Note: Currently all application-level and authentication events will return false. If you wish
1314+
/// Note: Currently all authentication events will return false. If you wish
13151315
/// to include additional no-site-info events, please add them here.
13161316
///
13171317
var shouldSendSiteProperties: Bool {
13181318
switch self {
1319-
// Application events
1320-
case .applicationClosed, .applicationOpened, .applicationUpgraded, .applicationInstalled, .watchAppOpened:
1321-
return false
13221319
// Authentication Events
13231320
case .signedIn, .logout, .openedLogin, .loginFailed,
13241321
.loginAutoFillCredentialsFilled, .loginAutoFillCredentialsUpdated, .loginEmailFormViewed, .loginMagicLinkOpenEmailClientViewed,

0 commit comments

Comments
 (0)