perawallet / pera-wallet

Source code for Pera Wallet, simply the best Algorand wallet.
https://perawallet.app
Other
178 stars 62 forks source link

Support for multisig wallets #56

Closed nhukc closed 2 years ago

nhukc commented 2 years ago

Feature request to add multisig wallet support.

A user should be able to create a multisig wallet by specifying public keys and should be able to sign transactions with their relevant public keys.

nhukc commented 2 years ago

Prototyping the multisig wallet creation experience for android.

image image image

I don't have the hardware required to work on the ios app. It'd be nice if someone could build the same experience on ios.

mateusz-sznura-ulam commented 2 years ago

Hello! What's the status of this one? Is there a support for signing multisig txns using Pera Wallet already? I'm trying to sign a multisig transaction using Pera Wallet, but I'm getting this error: Error: Invalid Input: The requested signer is not in the User Wallet (I'm using @walletconnect/client and the error is a response from algo_signTxn request with multisig params).

image

EDIT: Additional info provided.

nhukc commented 2 years ago

No, I haven't finished this. I'll see if I can make some time later this week to finish it up.

My code only applies to the Android app.

mateusz-sznura-ulam commented 2 years ago

And will it work (signing multisig transaction) if Pera Wallet is paired with Ledger?

nhukc commented 2 years ago

Yes, I intend to support hardware wallets.

maxcoto commented 2 years ago

Hey Nathan! Hope you're doing great. Is it there any ETA for this feature? Let me know if you need help with code, reviews or tests. We are willing to contribute as we are in the need for multisig signing app calls with a ledger device.

nhukc commented 2 years ago

The transition to pera-wallet made most of my code obsolete. Feel free to pick up this issue.

taylanpince commented 2 years ago

Thank you for this. multisig support is on our roadmap and we will have more updates on this later this year. Since this feature is captured on our roadmap (https://roadmap.perawallet.app/feature-requests), I am closing this issue.