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

Add rpc call z_validateaddress to validate zaddrs. #1748

Merged
merged 1 commit into from Nov 1, 2016

Conversation

Projects
None yet
5 participants
@bitcartel
Contributor

bitcartel commented Nov 1, 2016

Closes #1746.

@bitcartel bitcartel added this to the 1.0.1 milestone Nov 1, 2016

@bitcartel bitcartel self-assigned this Nov 1, 2016

@str4d

str4d approved these changes Nov 1, 2016

utACK

@ebfull

This comment has been minimized.

Show comment
Hide comment
@ebfull

ebfull Nov 1, 2016

Contributor

ACK great tests!

@zkbot r+

Contributor

ebfull commented Nov 1, 2016

ACK great tests!

@zkbot r+

@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Nov 1, 2016

Contributor

📌 Commit 4e16a72 has been approved by ebfull

Contributor

zkbot commented Nov 1, 2016

📌 Commit 4e16a72 has been approved by ebfull

@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Nov 1, 2016

Contributor

⌛️ Testing commit 4e16a72 with merge 77dcb00...

Contributor

zkbot commented Nov 1, 2016

⌛️ Testing commit 4e16a72 with merge 77dcb00...

zkbot pushed a commit that referenced this pull request Nov 1, 2016

zkbot
Auto merge of #1748 - bitcartel:master_1746_add_z_validateaddress, r=…
…ebfull

Add rpc call z_validateaddress to validate zaddrs.

Closes #1746.
@zkbot

This comment has been minimized.

Show comment
Hide comment
@zkbot

zkbot Nov 1, 2016

Contributor

☀️ Test successful - zcash

Contributor

zkbot commented Nov 1, 2016

☀️ Test successful - zcash

@zkbot zkbot merged commit 4e16a72 into zcash:master Nov 1, 2016

1 check passed

homu Test successful
Details

@bitcartel bitcartel removed the review needed label Nov 4, 2016

@Voluntary-zcash

This comment has been minimized.

Show comment
Hide comment
@Voluntary-zcash

Voluntary-zcash Nov 4, 2016

I've just installed v1.0.1 and I noticed that zcash-cli help doesn't list z_validateaddress

Voluntary-zcash commented Nov 4, 2016

I've just installed v1.0.1 and I noticed that zcash-cli help doesn't list z_validateaddress

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment