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

feat: [EUDIW-170] Upgrade react native to 75 #168

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

Conversation

luca-soda
Copy link
Collaborator

@luca-soda luca-soda commented Dec 9, 2024

List of Changes

This PR upgrades the project to React Native 0.75.4 from 0.72.14 to ensure compatibility with the latest features, fixes, and performance improvements provided by the framework.

Motivation and Context

This upgrade is essential to keep the project up-to-date with the latest React Native improvements

How Has This Been Tested?

Manual testing: compiled and ran Android.
I can't compile on iOS at the moment.

Checklist:

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

@luca-soda luca-soda requested a review from a team as a code owner December 9, 2024 16:09
@luca-soda luca-soda requested review from grausof and ale-mazz December 9, 2024 16:09
Copy link
Contributor

@LazyAfternoons LazyAfternoons left a comment

Choose a reason for hiding this comment

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

Can you please follow our convention for the PR title? There are also a few conflicts to be solved. Thanks.

@luca-soda luca-soda changed the title Update react native to 75 feat: [EUDIW-170] upgrade react native to 75 Dec 11, 2024
@LazyAfternoons LazyAfternoons changed the title feat: [EUDIW-170] upgrade react native to 75 feat: [EUDIW-170] Upgrade react native to 75 Dec 11, 2024
Comment on lines +5 to 9
"@pagopa/io-react-native-wallet": ["./src/index"],
"@pagopa/io-react-native-jwt": [
"./node_modules/@pagopa/io-react-native-jwt/src/index.ts"
]
},
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need this change? Why?

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.

3 participants