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

[Bug]: Wallet CLI does not allow full amount to be staked #2236

Closed
1 task done
PhilippGackstatter opened this issue Apr 26, 2024 · 2 comments
Closed
1 task done

[Bug]: Wallet CLI does not allow full amount to be staked #2236

PhilippGackstatter opened this issue Apr 26, 2024 · 2 comments
Assignees
Labels
bug Bug

Comments

@PhilippGackstatter
Copy link
Contributor

PhilippGackstatter commented Apr 26, 2024

Issue description

Wallet "alias": begin-staking 0xb8f540d46a9ebddc6bec85dafbe157f56d4aac033c3d216f115242a6512eb0e8 1000000000 2
Stronghold password: [hidden]
Begin staking for 0xb8f540d46a9ebddc6bec85dafbe157f56d4aac033c3d216f115242a6512eb0e8.
`invalid stakes amount`
2024-04-26 07:49:47 (UTC) wallet::wallet_cli                         ERROR `invalid stakes amount`

The account in question has exactly 1000000000 tokens. Only after lowering it to 100000 did it work.

Version

Branch from #2235.

Expected behaviour

I should be able to stake the full amount in an account.

Actual behaviour

The command failed with invalid stakes amount.

Can the issue reliably be reproduced?

Yes

Steps to reproduce the issue

  1. Create an account with account id id with X tokens.
  2. Run begin-staking id X 1.

Errors

See above.

Duplicate declaration

  • I have searched the issues tracker this issue and there is none
@Thoralf-M Thoralf-M added the bug Bug label Apr 26, 2024
@Thoralf-M Thoralf-M added this to the v2.0.0 (Q1 2024) milestone Apr 26, 2024
@DaughterOfMars
Copy link

This may be related to the missing requirement in the staking calls that is being fixed in #2235

@PhilippGackstatter
Copy link
Contributor Author

Sorry, I noted the wrong commit I was using. I actually used the branch from the #2235 PR for testing this, so it's still present there. Updated the description.

@Thoralf-M Thoralf-M self-assigned this Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug
Projects
Status: Done
Development

No branches or pull requests

3 participants