-
Notifications
You must be signed in to change notification settings - Fork 42
[MOB-12265] Configure start and end session #745
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
Changes from all commits
1bee2d6
0ba0b7f
24d5a70
be8998a
cbb547f
eb0dff5
efbda57
67e116b
bab2b35
fd42bac
433a089
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -14,6 +14,9 @@ import { IterableApi } from '../../core/classes/IterableApi'; | |||||
| export class IterableEmbeddedManager { | ||||||
| /** | ||||||
| * Whether the embedded manager is enabled. | ||||||
| * | ||||||
| * This is set through the `enableEmbeddedMessaging` flag in the | ||||||
| * `IterableConfig` class. | ||||||
| */ | ||||||
| private _isEnabled = false; | ||||||
|
|
||||||
|
|
@@ -41,4 +44,41 @@ export class IterableEmbeddedManager { | |||||
| getPlacementIds() { | ||||||
| return IterableApi.getEmbeddedPlacementIds(); | ||||||
| } | ||||||
|
|
||||||
| /** | ||||||
| * Starts a session. | ||||||
| * | ||||||
| * As session is a period of time when a user is on a screen or page that can | ||||||
|
||||||
| * As session is a period of time when a user is on a screen or page that can | |
| * A session is a period of time when a user is on a screen or page that can |
Copilot
AI
Nov 19, 2025
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.
The example shows static method call IterableEmbeddedManager.endSession(), but this should be an instance method call via Iterable.embeddedManager.endSession() to match the actual usage pattern in the SDK.
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.
JWT related things can be made a separate PR if it needs attention outside scope of Embedded Messaging.
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.
Second that!