Skip to content

Conversation

@sidhujag
Copy link
Member

@sidhujag sidhujag commented Jul 4, 2018

validateaddress should return the address scheme passed into validateaddress through "address" and not converting to Syscoin v3 address scheme. "saddress" is added to always give the S based address scheme.

validateaddress should return the address scheme passed into validateaddress through "address" and not converting to Syscoin v3 address scheme. "saddress" is added to always give the S based address scheme.
@willyko
Copy link

willyko commented Jul 4, 2018

i thought you want to call it v3address instead of saddress

@willyko willyko self-requested a review July 4, 2018 18:47
Copy link

@willyko willyko left a comment

Choose a reason for hiding this comment

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

tested ACK

@willyko
Copy link

willyko commented Jul 4, 2018

We should add that
< 3.0.6.1
validateaddress returns the following
address: S-address
btcaddress: 1-address

3.0.6.1 and up
validateaddress returns the following
address: input address
btcaddress: 1-address
saddress: S-address <- NEW

@sidhujag sidhujag merged commit fa6bfe9 into master Jul 4, 2018
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.

4 participants