fix: Forward impression events and products instead of null events#355
fix: Forward impression events and products instead of null events#355mmustafa-tse wants to merge 1 commit into
Conversation
PR SummaryLow Risk Overview When impressions are present, each list’s This aligns the Turbo Module path with the existing old architecture Reviewed by Cursor Bugbot for commit 557df1f. Bugbot is set up for automated code reviews on this repo. Configure here. |
| NSMutableDictionary *productDict = [[NSMutableDictionary alloc] init]; | ||
| if (productStruct.name()) productDict[@"name"] = productStruct.name(); | ||
| if (productStruct.sku()) productDict[@"sku"] = productStruct.sku(); | ||
| productDict[@"price"] = @(productStruct.price()); |
There was a problem hiding this comment.
Any reason price doesn't have a conditional whereas all the others do?
There was a problem hiding this comment.
@rmi22186 - I pretty much copied the same logic on for commerceEvent.products().has_value() in line 513, I can add the if statement for both to be on the safer side
rmi22186
left a comment
There was a problem hiding this comment.
Note this is new architecture that doesn't support it, not the old arch, the old arch supports, so the PR description is slightly off.
|
Also the iOS test app is not building. Take a look at that. |
Summary
A customer reported that Impression events sent from React Native SDK are being received as null in the UI, after further investigation it seems the iOS react native bridge was missing the logic to handle impressions in logCommerceEvents.
Testing Plan
Tested E2E by changing the code locally:

Master Issue
Closes https://rokt.atlassian.net/browse/SDKE-1210