Skip to content

Conversation

@jnewbery
Copy link
Contributor

Removes some deprecated code from the RPCs:

  • signrawtransaction was deprecated in 0.17 and removed in 0.18. A warning message was left in place to advise users to use signrawtransactionwithwallet and signrawtransactionwithkey. That warning can now be removed.
  • validateaddress had some functionality deprecated in 0.17 and removed in 0.18. The help text for that functionality was not removed in 0.18 and can be removed now.

signrawtransaction was deprecated in 0.17 and removed in 0.18. A warning
was left in place to tell users to migrate to using
signrawtransactionswithwallet or signrawtransactionwithkey. Remove the
warning now that it's been two releases since the method was removed.
@fanquake fanquake added this to the 0.19.0 milestone Mar 22, 2019
@jnewbery
Copy link
Contributor Author

One could argue that the second commit Remove deprecated functionality message from validateaddress help should be backported to 0.18, but I think it's so minor that it's not worth it.

@promag
Copy link
Contributor

promag commented Mar 22, 2019

@maflcko
Copy link
Member

maflcko commented Mar 22, 2019

ACK on the doc-backport

Copy link
Contributor

@ryanofsky ryanofsky left a comment

Choose a reason for hiding this comment

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

utACK 7b6616b. I would maybe change pr summary from "Remove deprecated rpc functionality" to "Remove deprecated rpc warnings" since "functionality" makes me think something more than error text is being removed.

@jnewbery jnewbery changed the title [rpc] Remove deprecated rpc functionality [rpc] Remove deprecated rpc warnings Mar 22, 2019
@jnewbery
Copy link
Contributor Author

@achow101
Copy link
Member

utACK 947f73c

1 similar comment
@maflcko
Copy link
Member

maflcko commented Mar 22, 2019

utACK 947f73c

@fanquake
Copy link
Member

utACK 947f73c

@promag
Copy link
Contributor

promag commented Mar 23, 2019

utACK 947f73c.

@laanwj laanwj merged commit 947f73c into bitcoin:master Mar 23, 2019
laanwj added a commit that referenced this pull request Mar 23, 2019
947f73c [docs] remove reference to signrawtransaction in the developer docs. (John Newbery)
7b6616b [rpc] Remove deprecated functionality message from validateaddress help (John Newbery)
839c3f7 [rpc] Remove signrawtransaction warning (John Newbery)

Pull request description:

  Removes some deprecated code from the RPCs:

  - signrawtransaction was deprecated in 0.17 and removed in 0.18. A warning message was left in place to advise users to use signrawtransactionwithwallet and signrawtransactionwithkey. That warning can now be removed.
  - validateaddress had some functionality deprecated in 0.17 and removed in 0.18. The help text for that functionality was not removed in 0.18 and can be removed now.

Tree-SHA512: 981678a697954ff2c392752e5a183b4b12c4eb94f55766ee1aa97a70d300668237db8fc5748c2772869d0155ba4a93e38817887b98160ee972a6f6ee94e3f7d9
vijaydasmp pushed a commit to vijaydasmp/dash that referenced this pull request Oct 16, 2021
947f73c [docs] remove reference to signrawtransaction in the developer docs. (John Newbery)
7b6616b [rpc] Remove deprecated functionality message from validateaddress help (John Newbery)
839c3f7 [rpc] Remove signrawtransaction warning (John Newbery)

Pull request description:

  Removes some deprecated code from the RPCs:

  - signrawtransaction was deprecated in 0.17 and removed in 0.18. A warning message was left in place to advise users to use signrawtransactionwithwallet and signrawtransactionwithkey. That warning can now be removed.
  - validateaddress had some functionality deprecated in 0.17 and removed in 0.18. The help text for that functionality was not removed in 0.18 and can be removed now.

Tree-SHA512: 981678a697954ff2c392752e5a183b4b12c4eb94f55766ee1aa97a70d300668237db8fc5748c2772869d0155ba4a93e38817887b98160ee972a6f6ee94e3f7d9
pravblockc pushed a commit to pravblockc/dash that referenced this pull request Nov 18, 2021
947f73c [docs] remove reference to signrawtransaction in the developer docs. (John Newbery)
7b6616b [rpc] Remove deprecated functionality message from validateaddress help (John Newbery)
839c3f7 [rpc] Remove signrawtransaction warning (John Newbery)

Pull request description:

  Removes some deprecated code from the RPCs:

  - signrawtransaction was deprecated in 0.17 and removed in 0.18. A warning message was left in place to advise users to use signrawtransactionwithwallet and signrawtransactionwithkey. That warning can now be removed.
  - validateaddress had some functionality deprecated in 0.17 and removed in 0.18. The help text for that functionality was not removed in 0.18 and can be removed now.

Tree-SHA512: 981678a697954ff2c392752e5a183b4b12c4eb94f55766ee1aa97a70d300668237db8fc5748c2772869d0155ba4a93e38817887b98160ee972a6f6ee94e3f7d9
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 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.

7 participants