Skip to content

[SDK-709]: Bump native SDK pins to Android 3.10.1 and iOS 6.7.5 - #898

Open
jferrao-itrbl wants to merge 3 commits into
masterfrom
feature/SDK-709-update-SDKs
Open

[SDK-709]: Bump native SDK pins to Android 3.10.1 and iOS 6.7.5#898
jferrao-itrbl wants to merge 3 commits into
masterfrom
feature/SDK-709-update-SDKs

Conversation

@jferrao-itrbl

@jferrao-itrbl jferrao-itrbl commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Pick up native JWT, WorkManager, and identity fixes without changing the RN public API.

📝 Summary

  • Pin-only native bump for SDK-709: Android com.iterable:iterableapi 3.6.2 → 3.10.1, iOS Iterable-iOS-SDK 6.6.7 → 6.7.5.
  • No JS, TurboModule, or bridge source changes. RN package version stays 3.1.0 so this branch can be used for regression before a versioned release.
  • Consumer-facing notes in CHANGELOG.md (Unreleased):
    • Android now pulls androidx.work:work-runtime:2.9.0 transitively (WorkManager since native 3.7.0). No app code change required.
    • On iOS, register(token:) (Iterable.registerDeviceToken) now requires a current user. A call before setEmail / setUserId does not send the register request; the JS method is fire-and-forget and does not surface that failure.
      Out of scope: JSON-only ack, InAppResponse.DEFER / resumeInAppDisplay(), unknown-user activation, IterableInAppDisplayMode, lastAuthTokenState, embedded sync success/failure callbacks, README version-mapping row (published 3.1.0 still shipped 3.6.2 / 6.6.7).
🎟️ Jira Ticket: SDK-709

🧪 How to test?

  • Gradle resolves com.iterable:iterableapi:3.10.1 (example/android debug runtime classpath)
  • pod install installs Iterable-iOS-SDK (6.7.5)
  • yarn test (585 passing), yarn typecheck, yarn lint
  • Example app compiles on iOS and Android
  • Smoke existing RN surface on iOS and Android:
    • Initialize
    • JWT authHandler (including re-init / Android retry-policy reflection)
    • In-app show/skip
    • Inbox session
    • Embedded sync
    • registerDeviceToken before vs after setEmail / setUserId

Pick up native JWT, WorkManager, and identity fixes without changing the RN public API.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jferrao-itrbl
jferrao-itrbl requested a review from a team as a code owner September 4, 2026 20:46
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Lines Statements Branches Functions
Coverage: 72%
71.92% (579/805) 61.22% (229/374) 67.18% (174/259)

@qltysh

qltysh Bot commented Sep 4, 2026

Copy link
Copy Markdown

Qlty


Coverage Impact

This PR will not change total coverage.

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

jferrao-itrbl and others added 2 commits September 7, 2026 09:42
Prerelease for native-pin regression testing before the final version cut.

Co-authored-by: Cursor <cursoragent@cursor.com>
Match the iOS SDK CI so xcodebuild does not fail with exit 70
when multiple iPhone 16 simulators are present.

Co-authored-by: Cursor <cursoragent@cursor.com>

@franco-zalamena-iterable franco-zalamena-iterable left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comments to address before merging, but looks ok so far

Comment thread CHANGELOG.md

- Bumped native SDK pins to Android [3.10.1](https://github.com/Iterable/iterable-android-sdk/releases/tag/3.10.1) and iOS [6.7.5](https://github.com/Iterable/swift-sdk/releases/tag/6.7.5). This is a native SDK bump, not new JS APIs.
- Android now pulls `androidx.work:work-runtime:2.9.0` as a transitive dependency (WorkManager replaced AsyncTask-based push handling in native 3.7.0). No app code change is required.
- On iOS, `register(token:)` (exposed as `Iterable.registerDeviceToken`) now requires a current user. Calling it before `setEmail` / `setUserId` no longer sends the register request; the native SDK reports failure instead. The JS method is fire-and-forget and does not surface that failure.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

from what i investigated the register(token:) already required a current user before this update, i don't think we need to add this to the changelog. I think the change is enableUnknownUserActivation default flipping from true to false

Comment thread package.json
{
"name": "@iterable/react-native-sdk",
"version": "3.1.0",
"version": "3.1.1-rc.1",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR body says version stays 3.1.0 but the diff sets 3.1.1-rc.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants