Concordium / concordium-client

A command line client to interact with the concordium-node
Apache License 2.0
8 stars 6 forks source link

Misleading message for `baker remove` #121

Closed concordium-cl closed 2 years ago

concordium-cl commented 2 years ago

Bug Description When removing a baker in the concordium-client, there are the following misleading messages, resulting from the fact that deregistering as baker is now the same as decreasing the stake to 0 and thereby below the baker threshold.

Warning: The staked amount (0.000000 CCD) is lower than the minimum baker stake threshold (1000.000000 CCD).
This transaction will most likely be rejected by the chain, do you wish to send it anyway [yN]: y
y
Warning: The new staked value appears to be lower than the amount currently staked on chain by this baker.

Expected Result When removing a baker/decreasing stake to 0, different warnings should be shown as when just decreasing stake to below baker thresold.

Actual Result Misleading warning that transaction will most likely be rejected.

Versions

concordium-cl commented 2 years ago
remove baker 2
concordium-cl commented 2 years ago

Similar issue fordelegator remove.

concordium-cl commented 2 years ago
delegating2
abizjak commented 2 years ago

Addressed in 4.0.1.