-
Notifications
You must be signed in to change notification settings - Fork 5
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
Problem: walletconnect 2.0 not supporting metamask android (fix #462) #464
Conversation
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #464 +/- ##
==========================================
+ Coverage 19.02% 19.03% +0.01%
==========================================
Files 24 24
Lines 3880 3877 -3
==========================================
Hits 738 738
+ Misses 3142 3139 -3 ☔ View full report in Codecov by Sentry. |
fixing audit issue need to upgrade ethers-rs to 2.0.11 |
b58eb0e
to
70b2303
Compare
1f1a219
to
2418453
Compare
checking ci |
fixing WalletConnectRust security issue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. left a few comments
Solution: fix send transaction tidy up send tx refactor remove comment tidy up remove debug print rename changelog change version update Cargo.lock update Cargo.lock fix fmt fix clippy update cargo.lock add system config fix mac build update defi-wallet-core update cargo.lock fix audit issue tidy up TOPIC tidy up from,to tidy up comment
d3f027c
to
37893a1
Compare
Solution: fix send transaction