Skip to content
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

addressTxos unconfirmed Tx #9

Closed
eek opened this issue Dec 30, 2017 · 3 comments
Closed

addressTxos unconfirmed Tx #9

eek opened this issue Dec 30, 2017 · 3 comments

Comments

@eek
Copy link

eek commented Dec 30, 2017

My latest transaction is an unconfirmed one on the https://vtc.blkidx.org/addressTxos/Vg8APdm2BhFbaZuTsXnG8Ny66k1YfnvurQ route

image

Is there a way we can add that this is an unconfirmed transaction and / or also add the number of confirmations it has? It's currently at 31 / 101

@gertjaap
Copy link
Collaborator

The height will be 0 for transactions in the memory pool.
The number of confirmations is the current last blockheight (you can fetch it from /sync) minus the height of the TX + 1.

@eek
Copy link
Author

eek commented Dec 30, 2017

Can we add the confirmation number that's available here: https://vtc.blkidx.org/getTransaction/832757b40a9e000bded60100df634496950f198b48a50a80336deb0b5f9cec55

to the addressTxos call instead of trying to compute it from the height? This will remove inconsistencies like the fact that the individual transaction api call doesn't return the height:

https://vtc.blkidx.org/getTransaction/832757b40a9e000bded60100df634496950f198b48a50a80336deb0b5f9cec55

EG:

{
blockhash: "56e46e47a4241426b9700eb69165eead57345e022697bcf457bad5dcb54398ea"
blocktime: 1514655340
confirmations: 14
hash: "90085747a5f40ac1a04cbef70c95a2dd9fca8b0efb620e175a7f718a5840cb0a"
hex: "010000000001010000000000…b5032506f6f6c5d00000000"
locktime: 0
size: 23727
time: 1514655340
txid: "832757b40a9e000bded60100…8a50a80336deb0b5f9cec55"
version: 1
vin: […]
vout: […]
vsize: 23700
}

@gertjaap
Copy link
Collaborator

gertjaap commented Feb 5, 2018

Moved here vertcoin-project/blockchain-indexer#1

@gertjaap gertjaap closed this as completed Feb 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants