Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Oct 7, 2022

I guess it can't hurt to return this for someone that wants to know it

@maflcko maflcko force-pushed the 2210-rpc-coinbase-scan- branch from 021a7f6 to fa08663 Compare October 7, 2022 13:05
Copy link
Contributor

@aureleoules aureleoules left a comment

Choose a reason for hiding this comment

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

ACK fa08663

Copy link
Contributor

@shaavan shaavan left a comment

Choose a reason for hiding this comment

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

ACK fa08663

Adding this option makes the resultant data consistent with the gettxout, which also displays the coinbase status for a single transaction.
Frame 1

@yancyribbens
Copy link
Contributor

Ack fa08663

@fanquake fanquake merged commit 1d277f4 into bitcoin:master Oct 12, 2022
@maflcko maflcko deleted the 2210-rpc-coinbase-scan-🐥 branch October 18, 2022 12:12
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 23, 2022
fa08663 rpc: Return coinbase flag in scantxoutset (MacroFake)

Pull request description:

  I guess it can't hurt to return this for someone that wants to know it

ACKs for top commit:
  aureleoules:
    ACK fa08663
  shaavan:
    ACK fa08663

Tree-SHA512: 04c554b3ed9877bab93ffcf0c1a4430cd41b30c5f4f3bf462a518fc8b3d68832dd85a29e81bd805eaa16e987856933d7a888a8c126f670bb2844bbd5ca1bf902
@bitcoin bitcoin locked and limited conversation to collaborators Oct 18, 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.

6 participants