Skip to content

Conversation

@theStack
Copy link
Contributor

@theStack theStack commented May 19, 2022

Including coinbase transactions in receivedby RPCs and adding the -deprecatedrpc=exclude_coinbase was done in PR #14707 (released in v23.0). For the next release v24.0, this configuration option can be removed.

@theStack theStack force-pushed the 202205-rpc-remove_deprecated_include_coinbase_option branch from f897b4b to 8469bad Compare May 19, 2022 14:35
@theStack theStack force-pushed the 202205-rpc-remove_deprecated_include_coinbase_option branch from 8469bad to a4703ce Compare May 19, 2022 14:43
Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

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

ACK a4703ce

@maflcko maflcko merged commit a7e3afb into bitcoin:master May 20, 2022
@theStack theStack deleted the 202205-rpc-remove_deprecated_include_coinbase_option branch May 20, 2022 12:10
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request May 28, 2022
…base` logic

a4703ce doc: add release notes about removal of the `deprecatedrpc=exclude_coinbase` (Sebastian Falbesoner)
ef0aa74 rpc: wallet: remove `-deprecatedrpc=exclude_coinbase` logic (Sebastian Falbesoner)

Pull request description:

  Including coinbase transactions in `receivedby` RPCs and adding the `-deprecatedrpc=exclude_coinbase` was done in PR bitcoin#14707 (released in v23.0). For the next release v24.0, this configuration option can be removed.

ACKs for top commit:
  fanquake:
    ACK a4703ce

Tree-SHA512: 97cd4e78501e64f678c78d2ebb5be5376688c023e34fced71dd24e432d27aa31a74b5483545f49ba0bdf48656d8b8b7bee74e3db26cf6daf112613f1caa4dfa4
@bitcoin bitcoin locked and limited conversation to collaborators May 20, 2023
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.

3 participants