leather-io / desktop

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

fix: pooling card amount, closes #1150 #1188

Closed kyranjamie closed 1 year ago

kyranjamie commented 1 year ago

Download the latest build

We used to get a value back from the stacking library that we no longer do (defaults to 0). Rather than showing the amount that the user allowed to delegate, we show the amount currently locked.

friedger commented 1 year ago

Locked balance does not look correct for delegation card.

When I use the library in a different project, I get the correct amount.

markmhendrickson commented 1 year ago

@friedger you're saying we do still get the proper "delegated amount" from the API, just need to fix a syntax error?

friedger commented 1 year ago

@markmhx Correct . remove the check for isBN

markmhendrickson commented 1 year ago

@kyranjamie let's get a fix in place with @friedger's guidance above and keep the meaning of "pooled" here consistent? If we weren't to have this data point, we'd need to change the label as well to prevent confusion (e.g. to "You have locked" or similar)

kyranjamie commented 1 year ago

Thanks @friedger, updated