You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When user does not have any T in his wallet we still display him that the minimal amount to stake is 40K:
When we put the correct value there it shows that the value should be less than or equal to 0:
We should display a proper error when user does not have any T (for example "You don't have that many T tokens in your wallet." cc @sashatanase@SorinQ).
We should also check what happens when users have some T but not enough to do stake (> 0 and < 40,000 T).
The content you are editing has changed. Please copy your edits and refresh the page.
It looks like for Goerli the minimum value is 0. On Mainnet it's 40k T. From what I see we also have a 40k minimum amount set in goerli-dapp-development packages. You can install them locally by running:
The error - "You don't have that many T tokens in your wallet" - should pop out if the value in the input exceeds tha value in the user's wallet. If this condition is met, then we should check if value in input is greater or equal to minimal value (which is 40k T). If not the we display the error that is implemented right now ("The value should be greater than or equal to <minimal_value>")
…x-improvements
Improve staking form user experience
Ref: #634
Added validation if user wallet is empty, simplified and made validation error
messages more descriptive, improved readability of validation function, disabled
form if wallet is not connected.
Some of the issues were also addressed in
#666, which was merged
into this PR.
When user does not have any T in his wallet we still display him that the minimal amount to stake is 40K:
When we put the correct value there it shows that the value should be less than or equal to 0:
We should display a proper error when user does not have any T (for example "You don't have that many T tokens in your wallet." cc @sashatanase @SorinQ).
We should also check what happens when users have some T but not enough to do stake (> 0 and < 40,000 T).
Pull Requests
The text was updated successfully, but these errors were encountered: