Bug: missing reloadOnDisconnect option passed in to WalletSDKRelay

[tindt-blockpass]

Describe the bug

I need to disable reload on disconnect behavior, so I set it to false in my code, but it didn't not work at all.

Steps

1. Connect to wallet.
2. Do some stuff (sign message).
3. Disconnect wallet.
   => The page auto reload even I set reloadOnDisconnect to false.

Expected behavior

=> The page should not reload.

Version

3.6.2

Additional info

No response

Desktop

• OS: macOS Ventura 13.1
• Browser: Chrome and Safari.

Smartphone

No response

[taycaldwell]

Cannot reproduce this behavior.

Do you have a demo/example where this behavior occurs?

Working demo: https://codesandbox.io/s/complete-uw25vs

  const coinbaseWallet = new CoinbaseWalletSDK({
    appName: APP_NAME,
    appLogoUrl: APP_LOGO_URL,
    reloadOnDisconnect: false
  });

[afmfe-iul]

@taycaldwell your example works, but I've added an example where the issue is happening, since no wallet is found (none is connected, you might have to disconnect yours from codesandbox), calling disconnect refreshes the page.

[bangtoven]

we have similar issues reported, so consolidating them into one for proper tracking. cc @vishnumad 🙏

[antonio-ivanovski]

Observing the same behavior when reloadOnDisconnect is provided as false

1. CoinbaseWalletSDK.disconnect() - reloads the page
2. CoinbaseWaleltProvider.disconnect() - doesn't do anything, doesn't disconnects the dapp form the wallet and it doesn't reload the page.
3. CoinbaseWaleltProvider.close() - reloads the page
4. Trigger disconnect from wallet - reloads the page

[antonio-ivanovski]

Added related issue (#802) to keep on mind when fixing this bug. Without it the DApp will continue thinking it is connected.

[vishnumad]

Fixed in #574