This brings up quite a few issues for one simple reason - we can connect two applications that use different networks and that breaks everything.
Example: when erc725Account is from L16 and DApp tries to use methods.owner().call() it expects the address of erc725Account to exist on testnet and sends requests to testnet RPC providers. From that line on everything stops working because there's no such address on testnet.
Also, there are a few other bug fixes like:
using channel instead of isDesktop for sendRequest function to decide how to send request;
in useWalletConnectV2 wrapped a few functions in try-catch.
github-actions[bot]
commented
1 year ago
See the video of how it works below.
With #117 DApp began to use only test net for its internal requests like:
This brings up quite a few issues for one simple reason - we can connect two applications that use different networks and that breaks everything. Example: when
erc725Accountis from L16 and DApp tries to usemethods.owner().call()it expects the address oferc725Accountto exist ontestnetand sends requests totestnetRPC providers. From that line on everything stops working because there's no such address ontestnet.Also, there are a few other bug fixes like:
channelinstead ofisDesktopforsendRequestfunction to decide how to send request;useWalletConnectV2wrapped a few functions in try-catch.https://github.com/lukso-network/universalprofile-test-dapp/assets/36865532/339045eb-5220-42cb-93fe-96404f917d86