Skip to content

Jwt/all - #721

Closed
lposen wants to merge 13 commits into
masterfrom
jwt/all
Closed

Jwt/all#721
lposen wants to merge 13 commits into
masterfrom
jwt/all

Conversation

@lposen

@lposen lposen commented Oct 6, 2025

Copy link
Copy Markdown
Contributor

🔹 JIRA Ticket(s) if any

✏️ Description

Please provide a brief description of what this pull request does.

lposen added 10 commits October 3, 2025 17:21
@lposen
chore: update Iterable API dependency to version 3.5.10 and add onAut…
80d67f6 
…hFailure event handling
@lposen
feat: implement onAuthFailure method in RNIterableAPIModule for new a…
3cb5d4c 
…rchitecture
@lposen
feat: enhance auth failure handling with detailed error reporting and…
f82e23b 
… new enum for failure reasons
@lposen
feat: add retry policy configuration to Iterable SDK with linear and …
8d01321 
…exponential backoff options
@lposen
feat: implement handling of auth failure
8ade24d
@lposen
chore: update Iterable-iOS-SDK dependency to version 6.6.1
ab554a1
@lposen
feat: add pauseAuthRetries method to RNIterableAPIModule for both arc…
70dbbc2 
…hitectures
@lposen
feat: refactor pauseAuthRetries method in ReactIterableAPI to use Ite…
e6e821e 
…rableAPI directly
@lposen
feat: add TODO.md file outlining new Iterable API features and config…
0470f8e 
…urations
@lposen
feat: enhance AppDelegate
d8cb608
@qltysh

qltysh Bot commented Oct 6, 2025 •
edited
Loading

Copy link
Copy Markdown

Diff Coverage: The code coverage on the diff in this pull request is 100.0%.

Total Coverage: This PR will decrease coverage by 4.4%.

File Coverage Changes
Path File Coverage Δ Indirect
src/api/NativeRNIterableAPI.ts 100.0
src/api/index.ts 100.0 √
src/core/classes/Iterable.ts -0.0 √
src/core/classes/IterableEdgeInsets.ts -100.0 √
src/core/classes/IterableUtil.ts -33.3 √
src/core/enums/IterableAuthFailureReason.ts 100.0
src/core/enums/IterableRetryBackoff.ts 100.0
src/core/hooks/useDeviceOrientation.tsx -100.0 √
src/core/types/IterableAuthFailure.ts 100.0
src/core/types/IterableRetryPolicy.ts 100.0
src/inApp/classes/IterableHtmlInAppContent.ts -100.0 √
src/inApp/classes/IterableInAppManager.ts -1.9 √
src/inbox/classes/IterableInboxDataModel.ts -2.2 √
src/inbox/components/HeaderBackButton.tsx -100.0 √
src/inbox/components/IterableInbox.tsx -1.2 √
🛟 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.

This is from Qlty Cloud, the successor to Code Climate Quality. Learn more.

@github-actions

github-actions Bot commented Oct 6, 2025 •
edited
Loading

Copy link
Copy Markdown
Lines Statements Branches Functions
Coverage: 47%
 46.98% (218/464) 22.04% (41/186) 40.38% (63/156)

@qltysh

qltysh Bot commented Oct 6, 2025 •
edited
Loading

Copy link
Copy Markdown

2 new issues

Tool Category Rule Count
qlty Structure Function with high complexity (count = 6): setupUserNotificationCenter 1
qlty Structure Function with many returns (count = 8): IterableAppProvider 1

This is from Qlty Cloud, the successor to Code Climate Quality. Learn more.

qltysh[bot]
qltysh Bot reviewed Oct 6, 2025
View reviewed changes
Comment thread example/ios/ReactNativeSdkExample/AppDelegate.swift
Comment thread src/core/enums/IterableAuthFailureReason.ts
Comment thread src/core/enums/IterableRetryBackoff.ts
Comment thread src/core/types/IterableAuthFailure.ts
Comment thread src/core/types/IterableRetryPolicy.ts
qltysh[bot]
qltysh Bot reviewed Oct 7, 2025
View reviewed changes
Comment thread example/ios/ReactNativeSdkExample/AppDelegate.swift
qltysh[bot]
qltysh Bot reviewed Oct 9, 2025
View reviewed changes
Comment thread example/ios/ReactNativeSdkExample/AppDelegate.swift
qltysh[bot]
qltysh Bot reviewed Oct 10, 2025
View reviewed changes
Comment thread example/ios/ReactNativeSdkExample/AppDelegate.swift
@lposen lposen added jwt do not merge Do not merge this branch!! Usually a test branch. labels Oct 14, 2025
Base automatically changed from new-arch/master to master November 3, 2025 20:11
qltysh[bot]
qltysh Bot reviewed Nov 3, 2025
View reviewed changes
Comment thread example/ios/ReactNativeSdkExample/AppDelegate.swift
UIApplication.shared.registerForRemoteNotifications()
}
}
}

@qltysh qltysh Bot Nov 3, 2025

Copy link
Copy Markdown

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 = 6): setupUserNotificationCenter [qlty:function-complexity]

@lposen lposen closed this Nov 19, 2025
@jferrao-itrbl
jferrao-itrbl deleted the jwt/all branch September 22, 2026 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@qltysh qltysh[bot] qltysh[bot] left review comments
Copilot code review Copilot
Copilot will automatically review once the pull request is marked ready for review

Assignees

No one assigned

Labels

do not merge Do not merge this branch!! Usually a test branch. jwt

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@lposen