-
Notifications
You must be signed in to change notification settings - Fork 38.7k
[rpc] Remove deprecated rpc warnings #15642
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] Remove deprecated rpc warnings #15642
Conversation
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.
This functionality was removed in v0.18.
|
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. |
|
ACK on the doc-backport |
ryanofsky
left a comment
There was a problem hiding this 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.
|
utACK 947f73c |
1 similar comment
|
utACK 947f73c |
|
utACK 947f73c |
|
utACK 947f73c. |
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
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
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
Removes some deprecated code from the RPCs: