Skip to content

Commit

Permalink
fix: braze push tracking not including url (#10678)
Browse files Browse the repository at this point in the history
fix braze push tracking not including url
  • Loading branch information
brainbicycle committed Aug 29, 2024
1 parent 105de57 commit 557ae4f
Showing 1 changed file with 15 additions and 2 deletions.
17 changes: 15 additions & 2 deletions ios/Artsy/App/ARAppNotificationsDelegate.m
Original file line number Diff line number Diff line change
Expand Up @@ -161,13 +161,26 @@ - (void)applicationDidReceiveRemoteNotification:(NSDictionary *)userInfo inAppli

- (void)receivedNotification:(NSDictionary *)notificationInfo;
{
[[AREmission sharedInstance] sendEvent:ARAnalyticsNotificationReceived traits:notificationInfo];
NSDictionary *normalizedInfo = [self normalizedNotificationInfo:notificationInfo];
[[AREmission sharedInstance] sendEvent:ARAnalyticsNotificationReceived traits:normalizedInfo];
[[SEGAnalytics sharedAnalytics] receivedRemoteNotification:notificationInfo];
}

- (NSDictionary *)normalizedNotificationInfo:(NSDictionary *)notificationInfo {
NSMutableDictionary *normalizedInfo = [notificationInfo mutableCopy];

if (notificationInfo[@"ab_uri"] != nil) {
normalizedInfo[@"url"] = notificationInfo[@"ab_uri"];
}

return normalizedInfo;
}

- (void)tappedNotification:(NSDictionary *)notificationInfo url:(NSString *)url;
{
[[AREmission sharedInstance] sendEvent:ARAnalyticsNotificationTapped traits:notificationInfo];

NSDictionary *normalizedInfo = [self normalizedNotificationInfo:notificationInfo];
[[AREmission sharedInstance] sendEvent:ARAnalyticsNotificationTapped traits:normalizedInfo];

NSDictionary *props = [self filteredProps:notificationInfo];
[[AREmission sharedInstance] navigate:url withProps:props];
Expand Down

0 comments on commit 557ae4f

Please sign in to comment.