Skip to content
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

wallet-api: Improvements #443

Merged
merged 2 commits into from
Jan 3, 2019
Merged

Conversation

j-mueller
Copy link
Contributor

  • AddressMap keeps track of TxOut instead of TxOutRef. This means we get the data script of the output as well
  • Add a payToScripts function in Wallet.API that lets us pay to multiple script addresses using our own funds

@j-mueller j-mueller requested review from nau and michaelpj January 2, 2019 12:59
@j-mueller j-mueller force-pushed the j-mueller/wallet-api-improvements branch from 6762a1f to 631272a Compare January 2, 2019 13:02
@j-mueller
Copy link
Contributor Author

hm, forgot to update the playground!

@j-mueller j-mueller force-pushed the j-mueller/wallet-api-improvements branch from 631272a to 7fd0075 Compare January 2, 2019 14:04
* AddressMap now keeps track of the entire `TxOut` value, rather than
  just the data `Amount`. In particular we get the output's data script
  and can re-use in another transaction.
@j-mueller j-mueller force-pushed the j-mueller/wallet-api-improvements branch from 7fd0075 to 72c94cb Compare January 3, 2019 08:15
@j-mueller j-mueller force-pushed the j-mueller/wallet-api-improvements branch from 72c94cb to c6c2ac8 Compare January 3, 2019 08:47
* We need a way to pay to multiple script outputs at the same time
@j-mueller j-mueller force-pushed the j-mueller/wallet-api-improvements branch from c6c2ac8 to e10b2b9 Compare January 3, 2019 09:48
@michaelpj michaelpj merged commit 92bc765 into master Jan 3, 2019
@kwxm kwxm deleted the j-mueller/wallet-api-improvements branch April 30, 2019 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants