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 #1

Open
gertjaap opened this issue Feb 5, 2018 · 2 comments
Open

addressTxos unconfirmed Tx #1

gertjaap opened this issue Feb 5, 2018 · 2 comments

Comments

@gertjaap
Copy link
Contributor

gertjaap commented Feb 5, 2018

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
Contributor Author

gertjaap commented Feb 5, 2018

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.

@gertjaap
Copy link
Contributor Author

gertjaap commented Feb 5, 2018

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
}

vertiond pushed a commit to vertiond/blockchain-indexer that referenced this issue Feb 24, 2021
Don't build restbed tests as they aren't needed for the docker image
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

1 participant