You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi, @daryl Collins we need your input on this PR since it’s adding a new function/method to the API of the wallet selector so we wanted to check with you.
This PR adds a new function addContractConnection which creates a new wallet connection (with a new prefix) through NAJ and adds a key by signing a transaction with the addKey action.
What they are attempting to do with this new function is to add support for multiple contracts.
This PR is related to the signInMulti but the approach is very different: #887
We left a comment in the signInMulti PR too: #811 (comment)
The text was updated successfully, but these errors were encountered:
Description
This is to review PR: wallet-selector: add function access keys to additional contracts by using multiple key stores
Kujtim Prenku
August 21, 2023 at 4:34 AM
Hi, @daryl Collins we need your input on this PR since it’s adding a new function/method to the API of the wallet selector so we wanted to check with you.
This PR adds a new function addContractConnection which creates a new wallet connection (with a new prefix) through NAJ and adds a key by signing a transaction with the addKey action.
What they are attempting to do with this new function is to add support for multiple contracts.
This PR is related to the signInMulti but the approach is very different:
#887
We left a comment in the signInMulti PR too:
#811 (comment)
The text was updated successfully, but these errors were encountered: