Lunie: Bonded validator error when bondedAtoms < 1

[fedekunze]

Description:

I bonded to a validator which I have 0 < steak < 1 bonded due to slashes showing that My Stake is NaN:

committedDelegations{
        cosmosaccaddr18thamkhnj9wz8pa4nhnp9rldprgant57ryzag7:  4.7e-9
...
}

Discussion:

What should we do with the leftover bonded atoms ?

[faboweb]

On the validator page, I showed < 0.01. Would that be sufficient?

[fedekunze]

yeah but the thing is that you can't do anything with < 0.01, not even with < 1

[faboweb]

You can undelegate up to 0.00000001 tokens I thought.

Federico Kunze [e-mail] schrieb am Fr., 28. Sep. 2018,
11:48:

yeah but the thing is that you can't do anything with < 0.01, not even
with < 1

—
You are receiving this because you commented.

Reply to this email directly, view it on GitHub
https://github.com/cosmos/voyager/issues/1369#issuecomment-425383939,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFmO2a61ZwyTVhzcVnRpqRxooQlqoEH2ks5uffBygaJpZM4W9G9P
.

[fedekunze]

I remember that we discussed to not be able to redelegate if the bonded amount was < 1

[faboweb]

You can undelegate and redelegate < 1 token (because we are undelegating and redelegating shares not tokens) but you can't delegate < 1 token. This is how it is currently implemented in the SDK.
We could be hiding validators with < 1 token delegation and hide decimals completely. This would already implement future SDK changes.

[fedekunze]

You can undelegate and redelegate < 1 token (because we are undelegating and redelegating shares not tokens)

ohh, I thought you were also talking about redelegation when you mentioned about not allowing users to put decimals on the ModalStake amount input.

We'll have to add a condition then on the submission button depending if it's delegation or redelegation to check those cases, change the input step and add some tests.

[faboweb]

Maybe this is all to complicated. Let's discuss in the Tuesday meeting.

[NodeGuy]

What did we decide in the meeting?

[faboweb]

We can't hide tokens for now. So we go with the < 0.01 term. This may already be solved.

[fedekunze]

Fixed on develop