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
Ratchet feature deprecation #6282
Conversation
0f8c293
to
00e0340
Compare
00e0340
to
9b6694a
Compare
The RPC method handler is left in as a tombstone, to redirect callers to the replacement method (as this is an upstream Bitcoin Core RPC method that users may expect to be present).
9b6694a
to
cbaf6b6
Compare
cbaf6b6
to
6ca3648
Compare
6ca3648
to
1671e40
Compare
1671e40
to
12ca9c9
Compare
12ca9c9
to
1de496f
Compare
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
BASE_ARGS = [ | ||
'-allowdeprecated=z_getnewaddress', | ||
'-allowdeprecated=z_getbalance', | ||
] | ||
TURNSTILE_ARGS = ['-experimentalfeatures', | ||
'-developersetpoolsizezero'] |
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.
Nit: use the same indentation as for BASE_ARGS
.
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, all comments are non-blocking.
These features were deprecated at least 3 minor releases ago. I found one mistake which was that `z_validateaddress` had not been placed behind the `addrtype` deprecated feature; this has been fixed.
1de496f
to
5cc15a8
Compare
Force-pushed to address @daira's comments. |
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
Post-hoc ACK for the changes since my last review. |
The deprecated feature was originally only applied to wallet code, but in zcash#6282 it was extended to a missing case in non-wallet code.
Closes #1555.
Closes #5513.
Closes #6318.