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

Expand warnings for baker/delegation commands #138

Open
shjortConcordium opened this issue May 6, 2022 · 0 comments
Open

Expand warnings for baker/delegation commands #138

shjortConcordium opened this issue May 6, 2022 · 0 comments
Labels
good first issue Good for newcomers [Type] Change Request Some visible functionality should be change.

Comments

@shjortConcordium
Copy link

shjortConcordium commented May 6, 2022

Description
Some places where we could warn the user, before they send a transaction, that will probably fail.

ConfigureBaker:

  • stake == 0 when sender is not a baker
  • when sender is already a delegator

When using the "old" baker commands when you are not a baker. (except add)

ConfigureDelegation:

  • stake == 0 when sender is not a delegator
  • when sender is already a baker
  • missing parameters when sender is not a delegator (like configureBaker does)

┆Issue is synchronized with this Jira Task by Unito

@shjortConcordium shjortConcordium added the [Type] Change Request Some visible functionality should be change. label May 6, 2022
@abizjak abizjak added the good first issue Good for newcomers label Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers [Type] Change Request Some visible functionality should be change.
Projects
None yet
Development

No branches or pull requests

2 participants