-
Notifications
You must be signed in to change notification settings - Fork 143
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pass in fee payer as public key #652
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! I created an issue to update the example contracts in the CLI once this PR lands and a new version if SnarkyJS is published to npm
.
Mina.transaction
API)AccountUpdate.fundNewAccount()
. Also, add an argument to fund multiple accounts and deprecate theinitialBalance
argument (can achieved in a cleaner way by calling.send()
on the returned account update)createSigned
,this.sign()
) along the waythis.sender
. Also available globally asMina.sender()
this.sender
in the dex examplesTODO: