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

Closes #1833. Format currency amounts in z_sendmany error message. #1838

Merged

Conversation

@bitcartel
Copy link
Contributor

bitcartel commented Nov 11, 2016

Improve coverage of possible error states from z_sendmany.
Refactor qa test for z_sendmany operations.

Improve coverage of possible error states from z_sendmany.
Refactor qa test for z_sendmany operations.
@bitcartel bitcartel added this to the 1.0.3 milestone Nov 11, 2016
@ebfull
Copy link
Contributor

ebfull commented Nov 13, 2016

ACK

@zkbot r+

@zkbot
Copy link
Collaborator

zkbot commented Nov 13, 2016

📌 Commit 9384e74 has been approved by ebfull

@zkbot
Copy link
Collaborator

zkbot commented Nov 13, 2016

Testing commit 9384e74 with merge 28d635c...

zkbot pushed a commit that referenced this pull request Nov 13, 2016
…ing, r=ebfull

Closes #1833.  Format currency amounts in z_sendmany error message.

Improve coverage of possible error states from z_sendmany.
Refactor qa test for z_sendmany operations.
@zkbot
Copy link
Collaborator

zkbot commented Nov 13, 2016

☀️ Test successful - zcash

@zkbot zkbot merged commit 9384e74 into zcash:master Nov 13, 2016
1 check passed
1 check passed
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.