Skip to content

Commit dfb4934

Browse files
committed
refactor: update templateId parameter type to Double in RNIterableAPI for consistency
1 parent e0d026e commit dfb4934

File tree

3 files changed

+8
-10
lines changed

3 files changed

+8
-10
lines changed

ios/RNIterableAPI/RNIterableAPI.mm

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ - (void)trackEvent:(NSString *)name dataFields:(NSDictionary *)dataFields {
155155
}
156156

157157
- (void)trackPushOpenWithCampaignId:(double)campaignId
158-
templateId:(NSNumber *_Nullable)templateId
158+
templateId:(double)templateId
159159
messageId:(NSString *)messageId
160160
appAlreadyRunning:(BOOL)appAlreadyRunning
161161
dataFields:(NSDictionary *)dataFields {
@@ -382,9 +382,8 @@ - (void)passAlongAuthToken:(NSString *_Nullable)authToken {
382382
}
383383

384384
RCT_EXPORT_METHOD(
385-
trackPushOpenWithCampaignId : (double)campaignId templateId : (
386-
NSNumber *_Nullable)templateId messageId : (NSString *)
387-
messageId appAlreadyRunning : (BOOL)
385+
trackPushOpenWithCampaignId : (double)campaignId templateId : (NSNumber *)
386+
templateId messageId : (double)messageId appAlreadyRunning : (BOOL)
388387
appAlreadyRunning dataFields : (NSDictionary *)dataFields) {
389388
[_swiftAPI trackPushOpenWithCampaignId:campaignId
390389
templateId:templateId

ios/RNIterableAPI/ReactIterableAPI.swift

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ import React
170170
@objc(trackPushOpenWithCampaignId:templateId:messageId:appAlreadyRunning:dataFields:)
171171
public func trackPushOpenWithCampaignId(
172172
campaignId: Double,
173-
templateId: NSNumber?,
173+
templateId: Double,
174174
messageId: String,
175175
appAlreadyRunning: Bool,
176176
dataFields: NSDictionary?
@@ -180,7 +180,7 @@ import React
180180

181181
IterableAPI.track(
182182
pushOpen: campaignId as NSNumber,
183-
templateId: templateId,
183+
templateId: templateId as NSNumber,
184184
messageId: messageId,
185185
appAlreadyRunning: appAlreadyRunning,
186186
dataFields: swiftDict)
@@ -388,7 +388,6 @@ import React
388388
ITBInfo()
389389
guard let message = IterableAPI.inAppManager.getMessage(withId: messageId) else {
390390
ITBError("Could not find message with id: \(messageId)")
391-
rejecter("", "Could not find message with id: \(messageId)", NSError(domain: "", code: 0, userInfo: nil))
392391
return
393392
}
394393
if let inAppDeleteSource = InAppDeleteSource.from(number: sourceNumber as NSNumber) {
@@ -416,8 +415,8 @@ import React
416415
templateId: Double
417416
) {
418417
ITBInfo()
419-
let finalCampaignId: NSNumber? = campaignId <= 0 ? nil : campaignId as NSNumber
420-
let finalTemplateId: NSNumber? = templateId <= 0 ? nil : templateId as NSNumber
418+
let finalCampaignId: NSNumber? = (campaignId as NSNumber).intValue <= 0 ? nil : campaignId as NSNumber
419+
let finalTemplateId: NSNumber? = (templateId as NSNumber).intValue <= 0 ? nil : templateId as NSNumber
421420
IterableAPI.updateSubscriptions(
422421
emailListIds,
423422
unsubscribedChannelIds: unsubscribedChannelIds,

src/api/NativeRNIterableAPI.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ export interface Spec extends TurboModule {
3939
): void;
4040
trackPushOpenWithCampaignId(
4141
campaignId: number,
42-
templateId: number | null,
42+
templateId: number,
4343
messageId: string,
4444
appAlreadyRunning: boolean,
4545
dataFields?: { [key: string]: string | number | boolean }

0 commit comments

Comments
 (0)