Skip to content
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

chore(IT Wallet): [SIW-1159] Add XState issuance machine scaffolding #5922

Merged
merged 264 commits into from
Jul 10, 2024

Conversation

mastro993
Copy link
Contributor

Warning

This PR depends on #5864

Short description

This PR adds the initial scaffolding of the IT Wallet activation and IT Wallet credential issuance XState machines.

State diagrams

List of changes proposed in this pull request

  • Added ItwEidIssuanceMachine with actions and actors stub implementations
  • Added ItwEidIssuanceMachineContext and Provider
  • Added ItwCredentialIssuanceMachine with actions and actors stub implementations
  • Added ItwCredentialIssuanceMachineContext and Provider
  • Wrapped ItwStackNavigator with the ItwEidIssuanceMachineProvider and ItwCredentialIssuanceMachineProvider

How to test

Static checks should pass.
From the IT Wallet playground, open the Discovery feature and check that machine events are correctly dispatched (navigation events).

CrisTofani and others added 30 commits January 19, 2024 12:28
Co-authored-by: Cristiano Tofani <cri.tofani@gmail.com>
@mastro993 mastro993 changed the base branch from master to wip-react-redux July 8, 2024 13:44
@mastro993 mastro993 requested a review from thisisjp as a code owner July 8, 2024 13:44
@mastro993 mastro993 changed the base branch from wip-react-redux to IOPLT-551-upgrade-to-rn-072 July 8, 2024 13:44
@mastro993 mastro993 force-pushed the SIW-1159-xstate-issuance-machine-scaffolding branch from e115a6c to df9cdcd Compare July 8, 2024 13:45
@mastro993 mastro993 force-pushed the SIW-1159-xstate-issuance-machine-scaffolding branch from df9cdcd to fdde218 Compare July 8, 2024 13:54
Base automatically changed from IOPLT-551-upgrade-to-rn-072 to master July 10, 2024 10:27
Copy link

dpulls bot commented Jul 10, 2024

🎉 All dependencies have been resolved !

@mastro993 mastro993 merged commit 04e83af into master Jul 10, 2024
13 checks passed
@mastro993 mastro993 deleted the SIW-1159-xstate-issuance-machine-scaffolding branch July 10, 2024 11:34
mastro993 added a commit that referenced this pull request Jul 16, 2024
…attestation retrieval (#5861)

> [!WARNING]
> This PR depends on #5922

## Short description
This PR introduces the wallet instance and attestation retrieval state
to the eID issuance machine.

## List of changes proposed in this pull request
- Added `@pagopa/io-react-native-integrity` dipendency
- Updated `@pagopa/io-react-native-wallet` to `^0.13.0`
- Moved cards and credentials catalog screen to the IT Wallet navigation
stack
- Added issuance reducer 
- Updated `itwEidIssuanceMachine` logic
  - Added `registerWalletInstance` and `getWalletAttestation` actors
- Added `WalletInstanceRegistration` and `WalletAttestationRetrieval`
substates to `WalletInitialization` state
 - Updated `ItwIdentificationModeSelectionScreen` copy

## How to test
In order to test this PR you need a **physical device** and the
[io-wallet local development
server](https://pagopa.atlassian.net/wiki/spaces/SIW/pages/1110900979/Come+utilizzare+il+backend+io-wallet+in+locale).
From the **IT Wallet playground**, go to the **Discovery** screen. You
should be able to go through the wallet init phase and land in the
identification screen

Co-authored by: @LazyAfternoons

---------

Co-authored-by: Mario Perrotta <mario.perrotta@pagopa.it>
Co-authored-by: Cristiano Tofani <cri.tofani@gmail.com>
Co-authored-by: Fabio Bombardi <16268789+shadowsheep1@users.noreply.github.com>
Co-authored-by: LazyAfternoons <lazyafternoons@outlook.it>
Co-authored-by: Damiano Plebani <damiano.plebani@pagopa.it>
Co-authored-by: Alice Di Rico <83651704+Ladirico@users.noreply.github.com>
Co-authored-by: LazyAfternoons <LazyAfternoons@users.noreply.github.com>
Co-authored-by: Andrea Piai <andrea.piai@pagopa.it>
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.

9 participants