-
Notifications
You must be signed in to change notification settings - Fork 12
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
#24 Update onboard dependencies and remove duplicates #44
#24 Update onboard dependencies and remove duplicates #44
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Pull Request Test Coverage Report for Build 5676848998
💛 - Coveralls |
e971219
to
fbf8148
Compare
Hi @brunomenezes, The Here for further reading on Walletconnect major changes: |
e1ca1f8
to
0d8aba2
Compare
0d8aba2
to
65a9da7
Compare
@dandheedge, |
f9164fa
to
3135ee5
Compare
Done |
@dandheedge, |
Upgrade to latest version for core, wallet-connect, ledger and injected-wallets. Configure ledger module to use WC v2.
Centralising env vars required for testing in one environment variables for testing tasks for apps and packages alike.
To highlight what is necessary. Check the README for Environment variables setup.
Update the docker image generation for rollups. Required changes given the upgrade in the wallet-connect version.
Add required configuration for metamask mobile wallet to work with WalletConnect v2.
Add information for next explorer release v3.6.0
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.
@dandheedge,
Ledger is now working, and we will keep it as an option. Another thing I pushed today was a necessary configuration for Metamask mobile to work with WalletConnect v2. Besides that, I am also already adding a changelog entry for release.
Update the onboard dependencies on the
packages/wallet
and remove fromapps/staking
from thepackage.json
.Fixes #24 #28