# of confirmations not calculated correctly #57

Open
opened 2019-02-25 02:04:58 +01:00 by tzarebczan · 5 comments
tzarebczan commented 2019-02-25 02:04:58 +01:00 (Migrated from github.com)

It shows 1 confirmation on block and transaction pages.

It shows 1 confirmation on block and transaction pages.
tzarebczan commented 2019-03-25 14:27:03 +01:00 (Migrated from github.com)

@akinwale / @nikooo777 you think you'd be able to take a look at this one soon? Community members are noticing it as well.

@akinwale / @nikooo777 you think you'd be able to take a look at this one soon? Community members are noticing it as well.
tiger5226 commented 2019-03-25 22:28:22 +01:00 (Migrated from github.com)

So we no longer keep old transactions that are not included in the leading chain. They are cleared. So technically we only have to take current height or max block height and subtract the height of the block in question. I have this in my @tzarebczan if they cannot get to it. I should have the ppv completed tonight. So I can probably make this one the next highest priority.

So we no longer keep old transactions that are not included in the leading chain. They are cleared. So technically we only have to take current height or max block height and subtract the height of the block in question. I have this in my @tzarebczan if they cannot get to it. I should have the ppv completed tonight. So I can probably make this one the next highest priority.
tzarebczan commented 2019-06-03 22:28:51 +02:00 (Migrated from github.com)

Looks like we had a fix, which worked for some, but if you look at https://explorer.lbry.com/address/bSSfLmehtc3PsDoj1ZDNmYkTbrNiTPj2CW the confirmations are not shown correctly on the top blocks.

Looks like we had a fix, which worked for some, but if you look at https://explorer.lbry.com/address/bSSfLmehtc3PsDoj1ZDNmYkTbrNiTPj2CW the confirmations are not shown correctly on the top blocks.
tiger5226 commented 2019-06-04 04:22:37 +02:00 (Migrated from github.com)

Also the block page does show the right confirmation. Its just this list apparently.

Also the block page does show the right confirmation. Its just this list apparently.
akinwale commented 2019-06-05 17:36:57 +02:00 (Migrated from github.com)

The old fix from 667097f35d (diff-84d5d8e3196d5767f5a47b5871946dddR598) can be adapted for this.

Essentially, we just need to obtain the max block height and then subtract the height of the corresponding transaction from the max height.

The old fix from https://github.com/lbryio/block-explorer/commit/667097f35d81e034b3130a5032d8f05461677b10#diff-84d5d8e3196d5767f5a47b5871946dddR598 can be adapted for this. Essentially, we just need to obtain the max block height and then subtract the height of the corresponding transaction from the max height.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/block-explorer#57
No description provided.