-
Notifications
You must be signed in to change notification settings - Fork 106
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
feat: [IOPLT-551] Upgrade react-native to 0.72 #5864
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mastro993
added a commit
that referenced
this pull request
Jul 10, 2024
…5922) > [!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 - IT Wallet activation machine: https://stately.ai/registry/editor/embed/393bb84d-c50f-4184-a882-0fcf0fff8def?machineId=6392ceb2-a47e-4484-8723-af34a8765b65 - IT Wallet credential issuance machine: https://stately.ai/registry/editor/embed/393bb84d-c50f-4184-a882-0fcf0fff8def?machineId=f54b1d6b-9163-49a4-a6a5-2a1f560001ba ## 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 I**T Wallet playground**, open the **Discovery** feature and check that machine events are correctly dispatched (navigation events). --------- Co-authored-by: Cristiano Tofani <[email protected]> Co-authored-by: Fabio Bombardi <[email protected]> Co-authored-by: LazyAfternoons <[email protected]> Co-authored-by: Damiano Plebani <[email protected]> Co-authored-by: Mario Perrotta <[email protected]> Co-authored-by: Alice Di Rico <[email protected]> Co-authored-by: LazyAfternoons <[email protected]> Co-authored-by: Andrea Piai <[email protected]>
CrisTofani
added a commit
that referenced
this pull request
Jul 23, 2024
> [!note] > Depends on #5864 ## Short description This PR sets Android API Level to 34 and upgrades `react-native-vision-camera` to solve compatibility issues ## List of changes proposed in this pull request - Sets Android API Level to 34 - Removes `vision-camera-code-scanner` - Updates `react-native-vision-camera` - Refactors the app components using the camera to scan Barcode ## How to test Test barcode code scanning to check everything is ok on both iOS and Android. Normally use the App on Android to check if the new API Level bump compromises anything --------- Co-authored-by: Fabio Bombardi <[email protected]> Co-authored-by: LazyAfternoons <[email protected]> Co-authored-by: Federico Mastrini <[email protected]> Co-authored-by: Damiano Plebani <[email protected]> Co-authored-by: Mario Perrotta <[email protected]> Co-authored-by: Alice Di Rico <[email protected]> Co-authored-by: LazyAfternoons <[email protected]> Co-authored-by: Andrea Piai <[email protected]>
dmnplb
added a commit
that referenced
this pull request
Aug 8, 2024
… + Dynamic header background (under the new Experimental Lab category) (#5954) > [!note] > This PR depends on: > * #5947 ## Short description This PR adds the latest version of the `react-native-skia` library and the new **Experimental Lab** section which includes the new **Dynamic header background** screen, built with the `skia` library ## List of changes proposed in this pull request - Add the `react-native-skia` library - Update `jest` config files to include `react-native-skia` references - Add the new **Experimental Lab** section - Add the new **Dynamic header background** as first experiment ### Preview https://github.com/pagopa/io-app/assets/1255491/802fee95-72ac-46d1-9a93-cdf8e5195b88 ### Acknowledgement We've successfully added this library thanks to the RN upgrade to the `0.72` version: - #5864 ## How to test 1. Launch the app in the local environment 2. Go to the **Experimental Lab** → **Dynamic header background** --------- Co-authored-by: LazyAfternoons <[email protected]> Co-authored-by: Cristiano Tofani <[email protected]> Co-authored-by: Federico Mastrini <[email protected]> Co-authored-by: Mario Perrotta <[email protected]> Co-authored-by: Fabio Bombardi <[email protected]> Co-authored-by: Alice Di Rico <[email protected]> Co-authored-by: LazyAfternoons <[email protected]> Co-authored-by: Andrea Piai <[email protected]>
dmnplb
added a commit
to pagopa/io-app-design-system
that referenced
this pull request
Aug 26, 2024
… the new `HStack` and `VStack` components (#303) >[!caution] > This PR depends on the following PR. **Do not merge the PR beforehand, otherwise it might break part of the UI**. > * pagopa/io-app#5864 ## Short description This PR refactors `Modules…` and `NumberPad` components to take advantage of the new `HStack` and `VStack` components. ## List of changes proposed in this pull request - Update `NumberPad` and `CodeInput` to replace the logic using `VSpacer` and `HSpacer` - Update all the `Modules…` components (both interactive and skeleton states) using `HStack` and `VStack` - Refactor `HStack` and `VStack` to apply some specific style properties (`alignItems`, `flexShrink`, etc…) ### Related PRs * #241 * #297 ## How to test 1. Launch the example app 2. First, check **Numberpad** screen 3. Second, check **Modules** screen
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Short description
This PR aims to upgrade react-native to 0.72 version and increment some of the chore dependencies
List of changes proposed in this pull request
react-native
versionjail-monkey
versionreact-native-reanimated
versionreact-native-pdf
versionreact-native-safe-area-context
versionreact-native-screens
versionreact-native-vision-camera
versionreact-native-popup-menu
react-native-sha256
How to test
Clean all projects build/cache folders, reinstall and build again