smartnode icon indicating copy to clipboard operation
smartnode copied to clipboard

Clearer error messages

Open PatriceVignola opened this issue 4 years ago • 1 comments

Some error messages can be hard to read when they're spelled out as separate words. For example, When calling rocketpool node set-withdrawal-address <address> at the moment when synced to mainnet you get something like this:

Could not get can set node withdrawal address

It's not immediately clear that this is supposed to be read as CanSetNodeWithdrawalAddress. I think replacing such messages by keeping the pascal case form of the function name could make it clearer to read (e.g. Could not get CanSetNodeWithdrawalAddress result). I can make the change, but I just wanted to know if I was the only one that had a hard time understanding what the exact error was at first.

PatriceVignola avatar Oct 02 '21 05:10 PatriceVignola

CanSetNodeWithdrawalAddress has confused a lot of folks as it seems, like me! But simple casing solution would solve this!

Glad @PatriceVignola pointed it out!

AayushCurious avatar Feb 07 '23 18:02 AayushCurious