Repository navigation
[SDK-706] Replace deep-link AsyncTask with SDK executors - #1094
Merged
franco-zalamena-iterable merged 5 commits intoOct 7, 2026
Merged
Conversation
rtlsilva
requested changes
Sep 21, 2026
franco-zalamena-iterable
force-pushed
the
feature/sdk-706-deeplink-redirect-executor
branch
from
September 23, 2026 09:22
9673361 to
8fe0331
Compare
franco-zalamena-iterable
force-pushed
the
feature/sdk-706-deeplink-redirect-executor
branch
from
September 23, 2026 12:37
8fe0331 to
8563c92
Compare
Contributor
Author
|
Stack review update after the rebase:
Check, BCIT, instrumentation, changelog, Java analysis, and CodeQL passed. I have rerun the unit job after its notification-image and request-ordering failures; the updated top branch passes the affected request tests locally. Ready for another review. |
Contributor
Author
|
Rerun follow-up: instrumentation remains green, and the unit rerun now fails only on the repository-wide external IterableNotificationTest.testNotificationImage assertion. The earlier request-ordering failure did not recur. |
rtlsilva
requested changes
Sep 24, 2026
franco-zalamena-iterable
force-pushed
the
feature/sdk-706-deeplink-redirect-executor
branch
from
October 2, 2026 14:03
8563c92 to
cb4710b
Compare
rtlsilva
self-requested a review
October 6, 2026 15:03
Base automatically changed from
feature/sdk-705-push-registration-executor
to
master
October 7, 2026 09:16
franco-zalamena-iterable
force-pushed
the
feature/sdk-706-deeplink-redirect-executor
branch
from
October 7, 2026 09:16
e82e1f1 to
e248fc8
Compare
franco-zalamena-iterable
deleted the
feature/sdk-706-deeplink-redirect-executor
branch
October 7, 2026 09:30
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 Summary
Moves Iterable deep-link redirect work off the process-wide AsyncTask queue while preserving callback behavior.
🎟️ Jira Ticket: SDK-706
📖 Description
This PR is stacked on #1093 (SDK-705).
🧪 How to test?
Passed:
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.