Skip to content

[SDK-706] Replace deep-link AsyncTask with SDK executors - #1094

Open
franco-zalamena-iterable wants to merge 1 commit into
feature/sdk-705-push-registration-executorfrom
feature/sdk-706-deeplink-redirect-executor
Open

franco-zalamena-iterable wants to merge 1 commit into
feature/sdk-705-push-registration-executorfrom
feature/sdk-706-deeplink-redirect-executor

Conversation

@franco-zalamena-iterable

@franco-zalamena-iterable franco-zalamena-iterable commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

📝 Summary

Moves Iterable deep-link redirect work off the process-wide AsyncTask queue while preserving callback behavior.

🎟️ Jira Ticket: SDK-706

📖 Description

  • Replaces the deep-link redirect AsyncTask with an SDK-owned serial executor.
  • Resolves redirects and attribution cookies in a focused Kotlin resolver.
  • Delivers client callbacks and attribution updates on the main thread, preserving their existing order.
  • Adds deterministic, public-flow tests for redirect outcomes, attribution, callback threading, operation ordering, protocol filtering, and handleAppLink behavior.
  • Replaces the ignored live-network handleAppLink test with MockWebServer coverage.

This PR is stacked on #1093 (SDK-705).

🧪 How to test?

Passed:

  • ./gradlew --no-daemon :iterableapi:testDebugUnitTest :iterableapi-ui:testDebugUnitTest
  • ./gradlew --no-daemon :iterableapi:lintDebug :iterableapi:checkstyle :iterableapi-ui:assembleDebug :app:testDebugUnitTest
  • Focused deep-link suite: 11 tests, 0 failures

The required standalone inbox-customization sample still fails on its existing Kotlin language-version mismatch: the sample uses Kotlin 1.8 while iterableapi-ui contains Kotlin 1.9 data object declarations.

🧾 Changelog

Updated the existing executor entry to include Iterable deep-link redirects and preserved main-thread callbacks.

📹 Loom recording if applicable

Not applicable.

🐞 Github Issues solved

None.

📚 Docs PR if applicable

Not applicable.

@franco-zalamena-iterable
franco-zalamena-iterable requested a review from a team as a code owner September 17, 2026 14:59
@franco-zalamena-iterable
franco-zalamena-iterable added this pull request to stack #1095 September 17, 2026 15:00
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.

1 participant