leather-io / desktop

Manage STX tokens and Stacking
https://leather.io/
191 stars 71 forks source link

[4.9.0] Bug: Incorrect pooling/stacked amount shown in Desktop Wallet #1150

Closed jdvanbru closed 1 year ago

jdvanbru commented 1 year ago

Bug found testing Hiro Wallet build undefined, 4.8.0.

markmhendrickson commented 1 year ago

Thanks for the report. Mind sharing a screenshot of what you see? Is the pooling amount specifically wrong, or is the amount of STX that are locked for Stacking wrong?

Possibly related: https://github.com/hirosystems/stacks-wallet/issues/1149

jdvanbru commented 1 year ago

Perhaps, the amount stacked and the amount pooled are/can be different?

The amount stacked appears to be correct, but shows 0 pooling.

314159265359879 commented 1 year ago

Hi @jdvanbru, no we expect those values to be the same. Although with the issue described in #1149 the value circled in green is correct and circled red is wrong due to an API issue. image I believe that is why Mark was asking if the "locked" value is wrong, or consistent with your expectation.

The cause of 1149 and likely this is an API issue being tracked here. Once that is resolved the issue in the wallet should be fixed too. The API issue is being tracked here: https://github.com/hirosystems/stacks-blockchain-api/issues/1520

314159265359879 commented 1 year ago

This issue is not resolved yet and popping up again now. Edu's comments about this in another issue https://github.com/hirosystems/desktop-wallet/issues/1155#issuecomment-1490535902

A user who delegated recently using the latest 4.9.0 Desktop wallet saw this after delegating STX at the end of cycle 56 (2 days to go for the start of 57) with the Desktop wallet. https://explorer.hiro.so/txid/0xb89490ec1680810fc9e8f14172f7e091f8139fb541daaeae5f2f1d8cd0ba3afe?chain=mainnet image

After using the lockstacks.com dapp with the extension instead the Desktop wallet shows: image and image

In the lockstacks.com dapp it shows like this: image

markmhendrickson commented 1 year ago

This is getting resolved by https://github.com/hirosystems/desktop-wallet/pull/1188