Skip to content

#7: pin expo-crypto version#8

Open
markovejnovic wants to merge 1 commit into
MobileWalletProtocol:masterfrom
markovejnovic:expo-crypto-bug
Open

#7: pin expo-crypto version#8
markovejnovic wants to merge 1 commit into
MobileWalletProtocol:masterfrom
markovejnovic:expo-crypto-bug

Conversation

@markovejnovic

Copy link
Copy Markdown

This PR fixes #7 by upgrading expo-crypto and expo-standard-web-crypto.

Running npx expo-doctor reports:

Running 15 checks on your project...
13/15 checks passed. 2 checks failed. Possible issues detected:
Use the --verbose flag to see more details about passed checks.

✖ Validate packages against React Native Directory package metadata
The following issues were found when validating your dependencies against React Native Directory:
  No metadata available: @mobile-wallet-protocol/client, @mobile-wallet-protocol/wagmi-connectors, expo-standard-web-crypto, viem, wagmi
Advice:
Update React Native Directory to include metadata for unknown packages. Alternatively, set expo.doctor.reactNativeDirectoryCheck.listUnknownPackages in package.json to false to skip warnings about packages with no metadata, if the warning is not relevant.

✖ Check that packages match versions required by installed Expo SDK
The following packages should be updated for best compatibility with the installed expo version:
  react-native@0.76.5 - expected version: 0.76.9
  react-native-screens@4.10.0 - expected version: ~4.4.0
Your project may not work correctly until you install the expected versions of the packages.
Found outdated dependencies
Advice:
Use 'npx expo install --check' to review and upgrade your dependencies.

npx expo install --check reports the same.

Testing

git clone https://github.com/markovejnovic/smart-wallet-expo-example.git
cd smart-wallet-expo-example && git checkout bbea53d1a0dbd87bf9e422547d124d27a80d0e43
npm i
npm run ios

The application builds successfully.

@markovejnovic

Copy link
Copy Markdown
Author

If this PR looks good, feel free to merge at your cadence.

Comment thread package.json
"@react-navigation/native": "^6.1.18",
"@tanstack/react-query": "^5.51.23",
"expo": "~52",
"expo-crypto": "~14.0.1",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

pnpm add @base-org/account @base-org/account-ui

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue Installing

4 participants