fabianbormann / cardano-peer-connect

This library aims to provide simple interfaces to implement CIP-0045 for dApps and wallets
Apache License 2.0
11 stars 4 forks source link

Bugfix: Autoconnect leads to missing wallet information in DApp #34

Closed marcuspuchalla closed 1 year ago

marcuspuchalla commented 1 year ago

In the current implementation, we offer the ability to automatically connect a wallet to a DApp after the connection has been established for the first time.

In doing so, the verifyConnection is not called anymore. However, this function was the only way for the DApp to get information about the connected wallet.

To fix this, I propose adding a second parameter to the onConnect function inside DAppPeerConnect.

This way, the DApp always gets the information about which wallet is connected.