/bitcoin.org

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Developer guide: details for gettransaction and listtransactions are wrong #1237

Closed
pinheadmz opened this Issue Feb 16, 2016 · 1 comment

Comments

Assignees

@wbnns wbnns

Labels
Dev Docs Merge Scheduled
Projects
None yet
Milestone
No milestone
3 participants
@pinheadmz @Mirobit @wbnns
@pinheadmz

pinheadmz commented Feb 16, 2016

See:

bitcoin/bitcoin#7532

result field blockindex is NOT "The block height of the block on the local best block chain which includes this transaction" it IS "the position of the transaction in the block"

Applies to gettransaction and listtransactions, there may be others

@pinheadmz pinheadmz referenced this issue in bitcoin/bitcoin Feb 16, 2016

Merged

Clarify description of blockindex #7541

MarcoFalke added a commit to MarcoFalke/bitcoin that referenced this issue Apr 27, 2016

@pinheadmz @MarcoFalke
Clarify description of blockindex 
see issues:

bitcoin-dot-org/bitcoin.org#1237

bitcoin#7532

Github-Pull: #7541
Rebased-From: 7eef1d0
52c1011

zander added a commit to zander/bitcoinclassic that referenced this issue Jun 16, 2016

@pinheadmz @zander
Clarify description of blockindex 
see issues:

bitcoin-dot-org/bitcoin.org#1237

bitcoin#7532

Github-Pull: #7541
Rebased-From: 7eef1d0
e4672f1

thokon00 added a commit to faircoin/faircoin that referenced this issue Jun 28, 2016

@pinheadmz @thokon00
Clarify description of blockindex 
see issues:

bitcoin-dot-org/bitcoin.org#1237

bitcoin#7532

Github-Pull: #7541
Rebased-From: 7eef1d0
74cebf6

@wbnns wbnns self-assigned this Dec 9, 2016

@wbnns wbnns added the Dev Docs label Dec 16, 2016

@Mirobit Mirobit referenced this issue Dec 25, 2016

Closed

"ListSinceBlock" docs wrong? #1457

@Mirobit
Contributor

Mirobit commented Jan 7, 2017

Thanks, addressed in 9c9edae

@wbnns wbnns added the Under Review label Jan 31, 2017

@wbnns wbnns added Merge Scheduled and removed Under Review labels Feb 9, 2017

@wbnns wbnns closed this Feb 16, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment