-
Notifications
You must be signed in to change notification settings - Fork 80
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
fix(deps): downgrade WalletConnect #4406
Conversation
1 build had no size change
Celo (test) 1.71.0 (136)No changes to report 🛸 Powered by Emerge Tools |
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.
Thanks!
This downgrade sounds ok for the short term. Though it is worrying if we can't upgrade with confidence.
We should chat about what we should do longer term.
events "^3.3.0" | ||
|
||
"@walletconnect/utils@2.10.0": | ||
version "2.10.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.
Same for this one?
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.
I was able to resolve this modifying the lock file directly, but I am not sure that's ideal. I believe @kathaypacific had to do something similar in the past with the WalletConnect dependencies.
Codecov Report
@@ Coverage Diff @@
## main #4406 +/- ##
=======================================
Coverage 84.90% 84.90%
=======================================
Files 701 701
Lines 25922 25922
Branches 3432 3432
=======================================
Hits 22008 22008
Misses 3854 3854
Partials 60 60 Continue to review full report in Codecov by Sentry.
|
@jeanregisser I wonder if we should clear out the sessions and pending requests in a redux migration. |
### Description It appears from [Sentry](https://valora-inc.sentry.io/discover/results/?cursor=0%3A250%3A0&field=title&field=release&field=environment&field=user.display&field=timestamp&name=Error%3A+Unable+to+find+WalletConnect+session+matching+topic+dfe8ce6bc3a89058f7a292b37d934e6f2bbe944a9bc15...&project=1250733&query=issue%3ACELO-MOBILE-EJR&sort=-timestamp&statsPeriod=90d&yAxis=count%28%29) that since v1.68.0 we've had increasing numbers of users hit issues with Wallet connect. This PR reverts upgrades to those dependencies to versions previously used. ### Test plan TBD ### Related issues N/A ### Backwards compatibility Yes
### Description Upgrades Wallet Connect dependencies to the latest versions: originally pinned in valora-inc#4406. ### Test plan - Tested locally on iOS and Android - Connect - Verify - Sign Transaction ### Related issues - Related: ACT-962 ### Backwards compatibility Yes ### Network scalability N/A
Description
It appears from Sentry that since v1.68.0 we've had increasing numbers of users hit issues with Wallet connect. This PR reverts upgrades to those dependencies to versions previously used.
Test plan
TBD
Related issues
N/A
Backwards compatibility
Yes