-
Notifications
You must be signed in to change notification settings - Fork 249
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
zcash_client_backend: Move change memos into the ChangeValue
components of Proposal
s
#1012
Conversation
Codecov ReportAttention:
📢 Thoughts on this report? Let us know!. |
b964e64
to
ad7117e
Compare
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.
utACK with suggestions.
5ef8687
to
f5b5849
Compare
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.
utACK
The wasi and clippy failures were due to failed downloads from crates.io, so I restarted them. |
f5b5849
to
73421ec
Compare
…ents of `Proposal`s. The existing API limited change outputs to having only a single memo repeated across each change output. This change makes it so that each proposed change output can have its own associated memo, and leaves it up to the input selector to determine how requested change memos are associated with change outputs.
Co-authored-by: Daira Emma Hopwood <[email protected]>
73421ec
to
bcea060
Compare
force-pushed to resolve conflicts after the merge of #1013 |
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.
utACK
The existing API limited change outputs to having only a single memo repeated across each change output. This change makes it so that each proposed change output can have its own associated memo, and leaves it up to the input selector to determine how requested change memos are associated with change outputs.