Rework transaction functions (Part 4)
> Follows https://git.duniter.org/clients/python/silkaj/issues/184 - [ ] tx: sources retrieved twice in `get_amount_from_pubkey()` and `get_sources()` - [ ] rename `get_amount_from_pubkey()` to `get_balance()` - [ ] handle `--allSources` with several recipients pubkeys: balance divided among recipients pubkeys - [x] e745bb95ed81dcbe985e3c185d01756cc1ff208f: [Refactor the one-liner condition](https://git.duniter.org/clients/python/silkaj/blob/dev/silkaj/tx.py#L80-97) - [ ] move to `transaction_confirmation()`? - [ ] Improve variable and function naming: - [ ] ~~`issuers` –> `issuer`~~: No, there could be many issuers, but it’s not implemented yet - [ ] `issuers` should be a list, even if it contains only one element for now. - [ ] camelize consistency for `outputBackChange` in code as a variable and as an option. Rename to `RecipientBackChange` - [ ] `output*` —> `recipient*`: - [ ] in the code - [x] in the cli: pass recipient pubkeys as arguments to be able to specify many without the `:` split - [ ] `curentUnitBase` –> `cur*r*entUnitBase` - [ ] `amount` –> `balance` - [ ] #193: Dataclass to access units in unit or in cent format - [ ] #119 Simplify confirmation message as now there is a recap chart before sending a transaction - [ ] rework, refactor this whole code
issue