WalletConnect / web3modal-react-native

Web3Modal React-Native SDK
https://web3modal.com
Apache License 2.0
46 stars 15 forks source link

WalletConnect V2 Metamask #61

Closed Z-Hayk closed 1 year ago

Z-Hayk commented 1 year ago

What problem does this new feature solve?

Hello. I am using your example but there is no Metamask in the list.

https://github.com/WalletConnect/react-native-examples -> dapps/v2Explorer

https://github.com/WalletConnect/web3modal-react-native/assets/122481542/c96faa2b-772c-45e9-a88b-ea9da931e8f6

System: OS: macOS 13.4 CPU: (8) x64 Intel(R) Core(TM) i5-1038NG7 CPU @ 2.00GHz Memory: 34.08 MB / 16.00 GB Shell: 3.2.57 - /bin/bash Binaries: Node: 18.13.0 - /usr/local/bin/node Yarn: 1.22.19 - /usr/local/bin/yarn npm: 8.19.3 - /usr/local/bin/npm Watchman: Not Found Managers: CocoaPods: 1.12.1 - /usr/local/bin/pod SDKs: iOS SDK: Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4 Android SDK: Not Found IDEs: Android Studio: 2022.2 AI-222.4459.24.2221.10121639 Xcode: 14.3/14E222b - /usr/bin/xcodebuild Languages: Java: Not Found npmPackages: @react-native-community/cli: Not Found react: 18.2.0 => 18.2.0 react-native: 0.71.8 => 0.71.8 react-native-macos: Not Found npmGlobalPackages: react-native: Not Found

"@babel/plugin-proposal-private-methods": "^7.18.6",
"@react-native-async-storage/async-storage": "1.18.1",
"@react-native-clipboard/clipboard": "1.11.2",
"@walletconnect/encoding": "1.0.2",
"@web3modal/react-native": "1.0.0-alpha.8",
"react": "18.2.0",
"react-native": "0.71.8",
"react-native-get-random-values": "1.9.0",
"react-native-modal": "13.0.1",
"react-native-svg": "^13.9.0"

Describe the solution you'd like

Would like to see on the list Metamask

ignaciosantise commented 1 year ago

Hey @Z-Hayk 👋 MetaMask will release a new version that supports WalletConnect V2 soon. Until they release this version, you won't be able to connect with MetaMask.

Z-Hayk commented 1 year ago

@ignaciosantise Thanks for the feedback.

Can you post links to merge requests or brunches here? To track their activities.

ignaciosantise commented 1 year ago

@Z-Hayk the v2 PR has been merged, and afaik they are testing it