lukso-network / universalprofile-test-dapp

Testing dApps for ERC725 and Universal Profiles.
https://up-test-dapp.lukso.tech/
17 stars 14 forks source link

fix: RPC responses didn't reach the UI; crashes, missing RPC methods; #115

Closed JeneaVranceanu closed 1 year ago

JeneaVranceanu commented 1 year ago
  1. Added all RPCs from Notion to EthereumProvider (the provider for the WalletConnect). As it appears it was missing requests like eth_getId which resulted in the following error:
Screenshot 2023-07-03 at 15 58 36
  1. Missing return! Looks like responses were discarded just because of the missing return.
  2. Swapped setupWeb3(provider as unknown as Provider) and setState('isConnected', true/false) to set provider first. After setting isConnected to true it triggers several background requests like web3.eth?.getChainId() but we have no provider and thus receive the following error:

Screenshot 2023-07-03 at 13 22 28

Swapping fixes the error.

github-actions[bot] commented 1 year ago

Deployed with Cloudflare Pages :cloud: :rocket: :ok:

JeneaVranceanu commented 1 year ago

Mainnet added to rpcMap. Ready to be deployed 🚀