[Hotfix - 1.6.1 🔥 ] - Fix for double QR code issue

[nenadV91]

Summary

Fixes #1787

Fixes issues with double QR code by updating @web3-react/walletconnect-connector to new version and adding fix for some additional issues that occur after the update like missing balances and wallet info.

To test

• We should test issues mentioned in [autoconnect] - wallet is not correctly connected and QR code remains #1787, [1.5.0] Prod: no wallet name in the Activity modal #1929 and [1.5.0] Prod: Can't load balances for a WC wallet in XDAI and Rinkeby #1930 with the steps to reproduce
• We should also test other functionality around WalletConnect and QR code

[github-actions]

• 🔭 GP Swap: Gnosis Protocol v2 Swap UI

[W3stside]

version name is outdated, please change this to reflect the current prod version and make sure everything is up to date. have you properly tested this?

can you provide detailed testing steps to make sure we catch any bugs and avoid what happened last time? @nenadV91

[elena-zh]

Hey @nenadV91 , the #1923 issue is still not fixed in this PR

[elena-zh]

@nenadV91 , I have found an issue with MM wallet when connected with the app using WC: I can't get a signing confirmation modal in the MM. I reported an issue for the #1976

In addition, I get stuck import tokens modal when I connect to MM in XDAI using WC. The modal does not disappear until I press on the import button

[nenadV91]

@elena-zh
The issue with the disconnect from the wallet side should be fixed now.
Regarding MM wallet, I just tried DAI->OWL, WETH->PAX on Rinkeby and it works fine.
I did notice some issues with the MM wallet last week also so could be that, maybe try again.

[elena-zh]

Hey @nenadV91 , changes LGTM besides an issue that is not related to this PR #1987 (co I can't test changes in Mainnet)
Also, connection is lost when change wallets, but we agreed to leave it as it is #1921 (comment)

[elena-zh]

Hey @nenadV91 , the #1987 issue seems to be fixed.
However, I have noticed that it is impossible to sign an order when connected to 1Inch wallet in this PR

I thought it was related to #1987, but it appeared that not. Could you please take a look into it?

Thanks

[anxolin]

From my side, code-wise seems alright to me. I would give my APPROVE if you get the approve from QA

I believe you have one unaddressed comment regarding 1inch

[nenadV91]

@anxolin I've posted about that in Slack https://gnosisinc.slack.com/archives/C025G521XQD/p1639403809006300, I don't have an iphone yet to debug this. I've tried browserstack but I think I need apple id to install new apps from the app store.

[W3stside]

should we close this? @nenadV91

[anxolin]

I thought this one was in production. can you confirm @nenadV91?

[elena-zh]

No, the issue is not in Prod. We could not release it due to 1Inch issue

[nenadV91]

As Elena said this is not in Prod since there is still issue with 1inch wallet

[anxolin]

@nenadV91, i change this into develop branch for whenever we resume it

[anxolin]

@nenadV91 any update on this?

[nenadV91]

@anxolin Nope, didn't work on it since we started working on claiming

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you think it shouldn't be closed, speak now or forever hold your peace.