Skip to content

Conversation

@mruddy
Copy link
Contributor

@mruddy mruddy commented Jan 13, 2018

"p2sh" in the help messages should have been "p2sh-segwit".

The messages before this patch:

help getnewaddress
"address_type" (string, optional) The address type to use. Options are "legacy", "p2sh", and "bech32". Default is set by -addresstype.

help getrawchangeaddress
"address_type" (string, optional) The address type to use. Options are "legacy", "p2sh", and "bech32". Default is set by -changetype.

@mess110
Copy link
Contributor

mess110 commented Jan 15, 2018

ACK 5f911c5

@maflcko maflcko added this to the 0.16.0 milestone Jan 15, 2018
@maflcko
Copy link
Member

maflcko commented Jan 15, 2018

ACK 5f911c5

@promag
Copy link
Contributor

promag commented Jan 16, 2018

ACK 5f911c5.

@maflcko maflcko merged commit 5f911c5 into bitcoin:master Jan 16, 2018
maflcko pushed a commit that referenced this pull request Jan 16, 2018
…d getrawchangeaddress

5f911c5 trivial: fix address_type help text of getnewaddress and getrawchangeaddress (mruddy)

Pull request description:

  "p2sh" in the help messages should have been "p2sh-segwit".

  The messages before this patch:

  `help getnewaddress`
  "address_type"   (string, optional) The address type to use. Options are "legacy", "**p2sh**", and "bech32". Default is set by -addresstype.

  `help getrawchangeaddress`
  "address_type"           (string, optional) The address type to use. Options are "legacy", "**p2sh**", and "bech32". Default is set by -changetype.

Tree-SHA512: 6dfc0bebe577995f5521b83a12854045ac3eda4e65c9b92fc581da4ee68ab1218e05af82f2154bb2640a0813c5f79e010cd9e5ada449494c8831b3757bda854c
@mruddy mruddy deleted the trivial1 branch January 18, 2018 12:24
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants