Skip to content

Commit df4b162

Browse files
authored
Merge pull request #1241 from OneSignal/fix/firebase_influence_open
Remove early return of trackReceivedEvent to allow for os_notification_influence_open event to be sent to Firebase
2 parents 0c1d008 + b5997c8 commit df4b162

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

iOS_SDK/OneSignalSDK/OneSignalCore/Source/OneSignalTrackFirebaseAnalytics.m

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -103,22 +103,11 @@ + (void)trackOpenEvent:(OSNotificationOpenedResult*)results {
103103
}
104104

105105
+ (void)trackReceivedEvent:(OSNotification*)notification {
106-
if (!trackingEnabled)
107-
return;
108-
109106
NSString *campaign = [self getCampaignNameFromNotification:notification];
110107
OneSignalUserDefaults *sharedUserDefaults = OneSignalUserDefaults.initShared;
111108
[sharedUserDefaults saveStringForKey:ONESIGNAL_FB_LAST_NOTIFICATION_ID_RECEIVED withValue:notification.notificationId];
112109
[sharedUserDefaults saveStringForKey:ONESIGNAL_FB_LAST_GAF_CAMPAIGN_RECEIVED withValue:campaign];
113110
[sharedUserDefaults saveDoubleForKey:ONESIGNAL_FB_LAST_TIME_RECEIVED withValue:[[NSDate date] timeIntervalSince1970]];
114-
115-
[self logEventWithName:@"os_notification_received"
116-
parameters:@{
117-
@"source": @"OneSignal",
118-
@"medium": @"notification",
119-
@"notification_id": notification.notificationId,
120-
@"campaign": campaign
121-
}];
122111
}
123112

124113
+ (void)trackInfluenceOpenEvent {

0 commit comments

Comments
 (0)