search

thencc / algonautjs

A front end friendly Algorand utility
MIT License
10 stars 1 forks source link

make a .defaultFromAddr field? #54

Closed spencercap closed 1 year ago

spencercap commented 1 year ago

when we switch over to externalizing the signing providers, this.account doesnt make much sense. the only real use is for populating .from fields w a default addr and the transactionSigner field in atomic txns.

might we introduce a this.setDefaultFromAddr method + associated field so that other (+ unsupported) wallets can use algonaut w ease?

spencercap commented 1 year ago

closing in favour of keeping Algonaut.account

Algonaut.account.address IS used as the default from address in transfer + app creates + calls etc

but note that Algonaut.account is of any-wallet's Account type (.addr, .walletId, .chain, .name), NOT the algosdk.Account type including the .sk

spencercap commented 1 year ago

FYI re-introduced Algonaut.account in: 99126ca4a7956e59ad7f355f28f11546068e23ef