Skip to content

Conversation

@franco-zalamena-iterable
Copy link

@franco-zalamena-iterable franco-zalamena-iterable commented Jan 21, 2026

🔹 Jira Ticket(s) if any

✏️ Description

Adding support for fully compose apps to have in-app messages without problems.

On IterableInAppDisplayer, we now detect if the activity is a ComponentActivity or FragmentActivity.

We now have an IterableInAppDialogNotification.kt for handling In-Apps for fully compose apps.

There is also InAppServices for handling the InApp actions, layout, etc., currently only implemented on the DialogNotification but created for sharing logic between the fragment and compose implementations

val loc = location ?: IterableInAppLocation.IN_APP

if (iterableApi != null) {
iterableApi.trackInAppOpen(messageId, loc)

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation Note

Invoking
IterableApi.trackInAppOpen
should be avoided because it has been deprecated.

Choose a reason for hiding this comment

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

@Ayyanchira What can be the replacement for that, since the same one is used in IterableInAppFragmentHTMLNotification:288

val loc = location ?: IterableInAppLocation.IN_APP

if (iterableApi != null) {
iterableApi.trackInAppClick(messageId, url, loc)

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation Note

Invoking
IterableApi.trackInAppClick
should be avoided because it has been deprecated.

Choose a reason for hiding this comment

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

@Ayyanchira What can be the replacement for that, since the same one is used in IterableInAppFragmentHTMLNotification:354

val loc = location ?: IterableInAppLocation.IN_APP

if (iterableApi != null) {
iterableApi.trackInAppClose(messageId, url, closeAction, loc)

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation Note

Invoking
IterableApi.trackInAppClose
should be avoided because it has been deprecated.

Choose a reason for hiding this comment

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

@Ayyanchira What can be the replacement for that, since the same one is used in IterableInAppFragmentHTMLNotification:370

@franco-zalamena-iterable franco-zalamena-iterable marked this pull request as ready for review January 21, 2026 16:05
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