feat: pass openChainModal even if connectionStatus is not connected

[hungdoansy]

Some DEXes often support multiple chains. Users tend to play around without connecting to the websites.

With the current value of ModalContext, it's not possible for users to switch chains without connecting wallets.

[vercel]

@hungdoansy is attempting to deploy a commit to the rainbowdotme Team on Vercel.

A member of the Team first needs to authorize it.

[hungdoansy]

Hi @magiziz, can you please review this PR?

[magiziz]

@hungdoansy Can you explain more why this change is needed ? Ideally we would want to open the chain modal after user has authenticated / connected their wallet, but lmk your what you think.

[hungdoansy]

@hungdoansy Can you explain more why this change is needed ? Ideally we would want to open the chain modal after user has authenticated / connected their wallet, but lmk your what you think.

I believe that people should be able to change the network to play with the webapp even tho they haven't connected yet.
Why?
Some people don't have wallet or they just don't want to connect to strange websites. We shouldn't prevent them playing around with the webapp.

How'd you think?

[0xartcro]

@hungdoansy Can you explain more why this change is needed ? Ideally we would want to open the chain modal after user has authenticated / connected their wallet, but lmk your what you think.

I believe that people should be able to change the network to play with the webapp even tho they haven't connected yet. Why? Some people don't have wallet or they just don't want to connect to strange websites. We shouldn't prevent them playing around with the webapp.

How'd you think?

#1806 Should fix that I believe

[magiziz]

@hungdoansy Can you explain more why this change is needed ? Ideally we would want to open the chain modal after user has authenticated / connected their wallet, but lmk your what you think.

I believe that people should be able to change the network to play with the webapp even tho they haven't connected yet. Why? Some people don't have wallet or they just don't want to connect to strange websites. We shouldn't prevent them playing around with the webapp.

How'd you think?

Right. That change will also cause users to switch chains without having their wallet connected which would result to unexpected behavior and like @0xartcro mentioned the PR #1806 should fix this behaviour.

[hungdoansy]

Thanks @magiziz @0xartcro for letting me know. Just took a look at the preview link. It's exactly what I needed. I'll close this PR now.