Skip to content

Conversation

lposen and others added 28 commits October 10, 2025 15:08
…-app' into loren/embedded/MOB-12261-add-embedded-manager-class
…-app' into loren/embedded/MOB-12261-add-embedded-manager-class
…to loren/embedded/MOB-12263-android-add-ability-to-sync-embedded-messages
@qltysh
Copy link

qltysh bot commented Oct 14, 2025

Qlty

Coverage Impact

⬆️ Merging this pull request will increase total coverage on master by 1.51%.

Modified Files with Diff Coverage (4)

RatingFile% DiffUncovered Line #s
Coverage rating: A Coverage rating: A
src/core/classes/IterableApi.ts100.0%
Coverage rating: A Coverage rating: A
src/core/classes/Iterable.ts100.0%
Coverage rating: A Coverage rating: A
src/core/classes/IterableConfig.ts100.0%
New file Coverage rating: A
src/embedded/classes/IterableEmbeddedManager.ts100.0%
Total100.0%
🚦 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.

@lposen lposen added the embedded Issues/PRs related to Embedded Messages label Oct 14, 2025
Base automatically changed from jwt/master to master December 11, 2025 04:36
lposen and others added 9 commits December 11, 2025 13:39
…-app' into loren/embedded/MOB-12261-add-embedded-manager-class
…to loren/embedded/MOB-12263-android-add-ability-to-sync-embedded-messages
…bedded-messages' into loren/embedded/MOB-12265-start-end-session
…embedded/MOB-12264-android-sync-and-get-messages
…-embedded-tab-in-example-app

[MOB-12260] create-embedded-tab-in-example-app
…bedded-manager-class

[MOB-12261] Add IterableEmbeddedManager class
…d-add-ability-to-sync-embedded-messages

[MOB-12263] android-add-ability-to-get placements
…embedded/MOB-12264-android-sync-and-get-messages
…end-session

[MOB-12265] Configure start and end session
…d-sync-and-get-messages

[MOB-12264] Add sync messages and get messages to android
} catch (JSONException e) {
IterableLogger.e(TAG, e.getLocalizedMessage());
promise.reject("", "Failed to fetch messages with error " + e.getLocalizedMessage());
}
Copy link

Choose a reason for hiding this comment

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

Function with high complexity (count = 11): getEmbeddedMessages [qlty:function-complexity]

'Notification permission not required for this Android version.'
);
}
}
Copy link

Choose a reason for hiding this comment

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

Function with high complexity (count = 15): requestNotificationPermission [qlty:function-complexity]

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

Labels

embedded Issues/PRs related to Embedded Messages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants