Skip to content

feat(example): [SIW-2798] Add new CIE SDK in example app #275

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

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

mastro993
Copy link
Contributor

@mastro993 mastro993 commented Aug 14, 2025

List of Changes

  • Removed @pagopa/react-native-cie package
  • Added @pagopa/io-react-native-cie package
  • For Android
    • added android.permission.NFC in the AndroidManifest.xml
    • removed manual linking from MainApplication.kt, that was previously required by the legacy SDK
    • removed dependencies from app/build.gralde, that was previously required by the legacy SDK
  • Refactored CIE flow UI logic:
    • Added CieWebView, CieAuthenticationWebview and CieAuthorizationWebview components
    • Added useCie hook to handle all the CIE read process

Motivation and Context

This PR completely removes the legacy CIE SDK and implements the new one.

How Has This Been Tested?

Obtain a PID using CIE+PIN flow and ensure everything works correctly on both iOS and Android

Screenshots (if appropriate):

iOS Android
ScreenRecording_08-14-2025.10-55-25_1.MP4
document_5832475763021978260.mp4

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@mastro993 mastro993 self-assigned this Aug 14, 2025
@mastro993 mastro993 requested a review from a team as a code owner August 14, 2025 09:38
@github-actions github-actions bot added the feat label Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant