Skip to content
This repository has been archived by the owner on Jul 31, 2021. It is now read-only.

WalletConnect link not working, js site crashes #124

Closed
testacode opened this issue Jan 7, 2021 · 15 comments · May be fixed by #191
Closed

WalletConnect link not working, js site crashes #124

testacode opened this issue Jan 7, 2021 · 15 comments · May be fixed by #191
Labels
bug Something isn't working

Comments

@testacode
Copy link

testacode commented Jan 7, 2021 •
edited
Loading

Bug Description
When navigating to exchange.pancakeswap.finance with latest version of Chrome or Safari (from OSX Big Sur also latest version) and trying to connect with Walletconnect after reading the QR with my iPhone 12 (iOS latest version) and also latest version of TrustWallet the site crashes. Below it's the screenshot.
image

Steps to Reproduce

  1. Go to exchange.pancakeswap.finance.
  2. Click on "connect" button (upper right).
  3. Select "WalletConnect".
  4. Scan QR with TrustWallet
  5. Site JS crashes.
    ...

Expected Behavior
The expected behavior should be that the app won't crash and let me stake or trade my tokens.

Video
uxjDVzMMAI
@testacode testacode added the bug Something isn't working label Jan 7, 2021
@testacode
Copy link
Author

It's working now. It just wasn't a couple of hours ago!! You can close this!

@testacode
Copy link
Author

It's happening on the exchange now

@testacode testacode reopened this Jan 7, 2021
@testacode testacode changed the title Walletconnect not working at all, site crashes WalletConnect link not working, js site crashes Jan 7, 2021
@xandercakes
Copy link

xandercakes commented Jan 8, 2021 •
edited
Loading

The same issue occurs when trying us WalletConnect and SafePal on exchange.pancakeswap.finance. Reproduced the above issue with WalletConnect using both SafePal and Trust Wallet (Android Version 11, Brave and Firefox).

Console report:

console-export-2021-1-8_2-47-51.txt

@NinZine
Copy link

NinZine commented Jan 18, 2021

Reproduced both on Android and iOS with Trust Wallet.

TypeError: Cannot read property 'address' of undefined
    at main.bd05afab.chunk.js:1
    at Sb (main.bd05afab.chunk.js:1)
    at Rb (main.bd05afab.chunk.js:1)
    at no (4.b8eba4b2.chunk.js:1)
    at Jo (4.b8eba4b2.chunk.js:1)
    at Xs (4.b8eba4b2.chunk.js:1)
    at Tu (4.b8eba4b2.chunk.js:1)
    at Cu (4.b8eba4b2.chunk.js:1)
    at xu (4.b8eba4b2.chunk.js:1)
    at vu (4.b8eba4b2.chunk.js:1)

@toppagc
Copy link

toppagc commented Feb 3, 2021

This is still an issue. It’s 100% reproducible. I can’t connect to the exchange at all. Please fix this.

@trumhemcut
Copy link

me too.

@taasaa
Copy link

taasaa commented Feb 9, 2021

Happens to me too. Any idea when it will be addressed? I can't use pancake with my wallets

@testacode
Copy link
Author

testacode commented Feb 9, 2021 •
edited
Loading

Happens to me too. Any idea when it will be addressed? I can't use pancake with my wallets

Actually you can. You can use any network compatible wallet with a web3 browser. For example with SafePal's App, just use the wallet with your coins by importing the seed phrase or password into SafePal and you're good to go.

@adaamz
Copy link

adaamz commented Feb 14, 2021

same here, when I proposed myself to fix this bug i was banned on Telegram...

@RabbitDoge
Copy link
Contributor

same here, when I proposed myself to fix this bug i was banned on Telegram...

If you want to help solving this issue, please propose a solution here.

@adaamz
Copy link

adaamz commented Feb 17, 2021

Looks like MetaMask doesnt work too. Should I create new issue?

@rapinto
Copy link

rapinto commented Feb 17, 2021 •
edited
Loading

same problem using "wallet connect" from Trust Wallet iOS mobile App to connect on the web site from my PC with brave browser.

@Johann-S Johann-S mentioned this issue Feb 17, 2021
Open
4 tasks
@vitorperim
Copy link

Same issue trying to use the exchange. I tried to connect using the "wallet connect" from Trust Wallet iOS mobile. I tried it from my PC and my cellphone. Both the same problem

@testacode
Copy link
Author

This still happening says a lot about the development path of the team. Too bad.

@RabbitDoge
Copy link
Contributor

Solved

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: unknown chain id for weth contract Johann-S/pancake-swap-interface
10 participants
@taasaa @NinZine @rapinto @testacode @adaamz @trumhemcut @toppagc @RabbitDoge @xandercakes @vitorperim