safe-global / safe-react-apps

A repository for applications for Safe Web Interface
MIT License
98 stars 73 forks source link

[WalletConnect] Enable connection without considering chain #763

Closed dasanra closed 11 months ago

dasanra commented 11 months ago

Overview

After few months, even now requiredChains is an optional parameter, many connection providers didn't adapt. This makes impossible to connect some dApps to the Safe when using WalletConnect. Also a drawback for these providers to prevent adding this compatibility is because many other Wallet apps require explicitly the property, making the migration complex. https://github.com/wagmi-dev/wagmi/issues/3027

Many providers accept a list of chains and always set the first one as required, even the others are set as optional. As the Safe can't guarantee that the required is always available the connection is automatically rejected.

Requirements

Designs

Links

johannesmoormann commented 11 months ago

will be addressed with https://github.com/safe-global/safe-wallet-web/pull/2535