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

[RPC] fix mintzerocoin documentation. #795

Merged
merged 1 commit into from
May 4, 2021
Merged

[RPC] fix mintzerocoin documentation. #795

merged 1 commit into from
May 4, 2021

Conversation

philmb3487
Copy link
Contributor

problem

The help documentation for the method "mintzerocoin" is confusing and/or inaccurate.

how to solve

I have added in the missing parameter to the help text, and fixed up the examples, and reordered the arguments to match reality, and fixed the result list to match reality.

Copy link
Collaborator

@CaveSpectre11 CaveSpectre11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK ecc7e97

@CaveSpectre11 CaveSpectre11 added Coin Type: Zerocoin Specifically related to zerocoin Component: RPC Related to the console commands themselves. labels May 4, 2021
@CaveSpectre11 CaveSpectre11 added the Tag: Waiting For Code Review Waiting for code review from a core developer label May 4, 2021
Copy link
Collaborator

@codeofalltrades codeofalltrades left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK ecc7e97

@codeofalltrades codeofalltrades added Code Review: Passed and removed Tag: Waiting For Code Review Waiting for code review from a core developer labels May 4, 2021
@codeofalltrades codeofalltrades merged commit 9f0212d into Veil-Project:master May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Review: Passed Coin Type: Zerocoin Specifically related to zerocoin Component: RPC Related to the console commands themselves.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants