-
Notifications
You must be signed in to change notification settings - Fork 81
[MOB-11903] consent logging refactoring #938
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
bug fixes with double values having trailing zeros
… statements for track functions
Minmatch and single item matches done with test cases
…ta2-release [MOB-10367] update pointers and changelog
# Conflicts: # CHANGELOG.md # Iterable-iOS-AppExtensions.podspec # Iterable-iOS-SDK.podspec # swift-sdk.xcodeproj/project.pbxproj # swift-sdk/Core/Constants.swift # swift-sdk/Internal/InternalIterableAPI.swift # swift-sdk/Internal/IterableUserDefaults.swift # swift-sdk/Internal/Utilities/LocalStorage.swift # swift-sdk/SDK/IterableAPI.swift # swift-sdk/SDK/IterableConfig.swift # tests/common/MockLocalStorage.swift
Co-authored-by: Evan Greer <evan.greer@evan.greer> Co-authored-by: Joao Dordio <joaodordio@icloud.com>
Co-authored-by: Megha <megha.pithadiya@iterable.com> Co-authored-by: Evan Greer <evan.greer@evan.greer>
# Conflicts: # Iterable-iOS-AppExtensions.podspec # Iterable-iOS-SDK.podspec # swift-sdk/Internal/InternalIterableAPI.swift # swift-sdk/SDK/IterableAPI.swift
Co-authored-by: Evan Greer <evan.greer@evan.greer>
# Conflicts: # Iterable-iOS-AppExtensions.podspec # Iterable-iOS-SDK.podspec # swift-sdk.xcodeproj/project.pbxproj # swift-sdk/Internal/AuthManager.swift # swift-sdk/SDK/IterableAPI.swift
…iOS SDK (#922) Co-authored-by: Joao Dordio <joaodordio@icloud.com> Co-authored-by: Evan Takeo Kanaiaupuni Greer <56953678+evantk91@users.noreply.github.com> Co-authored-by: Evan Greer <evan.greer@evan.greer>
Co-authored-by: Evan Greer <evan.greer@evan.greer>
Co-authored-by: Evan Greer <evan.greer@evan.greer>
Co-authored-by: Evan Greer <evan.greer@evan.greer>
…etes" This reverts commit 62bcd84.
Co-authored-by: Evan Greer <evan.greer@evan.greer>
).onSuccess { _ in | ||
ITBInfo("Pending consent tracked successfully") | ||
}.onError { error in | ||
ITBError("Failed to track pending consent: \(error)") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like we have discussed, it would be good to have a retry mechanism here that tries to track the consent a second time and then fails definitely.
} | ||
|
||
// Clear the pending consent data | ||
pendingConsentData = nil |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be good to call the clearPendingConsent() method here instead.
🔹 Jira Ticket(s)
✏️ Description