Modify naming flags

[bneiluj]

Delegate, secret are a bit confusing in the current description:

We should modify delegate to bakerAddress so it's very clear that it's the baker's address and not the payout's wallet. Also, the fact that there is no need to specify the Payout Wallet Public Key because of passing the PK should be made clear to the user by using secretPayoutWallet.

Naming proposal:

i). bakerAddress: Baker's address, the node that is running the Tezos Bakery.
ii). delegators: the different addresses that are delegating to the Baker's address.
iii). payoutWallet & secretPayoutWallet: the wallet that is sending payouts to delegator addresses. We can use secretPayoutWallet for "encrypted secret key of the wallet paying (e.g. --secret=)"

To look have something like that:

./payman payout \
    --bakerAddress=tz3gN8NTLNLJg5KRsUU47NHNVHbdhcFXjjaB \
    --secretPayoutWallet=edesk1Qx5JbctVnFVHL4A7BXgyExihHfcAHRYXoxkbSBmKqP2Sp92Gg1xcU8mqqu4Qi9TXkXwomMxAfy19sWAgCm \
    --password=abcd1234 \
    --cycle=160 \
    --node=127.0.0.1 \
    --port=8732 \
    --fee=0.05 

[DefinitelyNotAGoat]

@bneiluj I'm going to have to disagree with you on these changes. I want to keep the flags uniform with the official Tezos Documentation. Tezos refers to bakers as Delegates..

I also believe the descriptions are good, but if there's something you believe is missing in them, I could update those.

-d, --delegate string        public key hash of the delegate that's paying out (e.g. --delegate=<phk>)
-s, --secret string          encrypted secret key of the wallet paying (e.g. --secret=<sk>)

Thanks for your feedback!

[DefinitelyNotAGoat]

Also @bneiluj by the looks of it you may be using an old version on payman. I would recommend you update to v1.0.3!

[bneiluj]

@DefinitelyNotAGoat Ok perfect. Let's just update the description. It makes sense to me.