- Make known that non-zero returns are fatal errors
- Return success on success (iadd | manual)
- Add iadd to `edit` usage help
- Remove outdated comments
A recent combination of Coinbase and ccxt changes will now result in
authentication errors (server error 401) because of docker-finance's
coinbase implementation (specifically, direct calls to internal ccxt).
docker-finance's previous usage of request() meant hooking into the
underlying ccxt API and not the end-user public-facing API (publicly
documented - not necessarily the "public" vs "private" APIs provided by
exchanges, as seen in ccxt documentation).
This direct usage of request() was obviously never recommended but it
was always easier to interact with to get to the underlying raw object
(at least at the time of original docker-finance coinbase writing).
Since it appears that ccxt has refactored the authentication mechanism
related to request(), this commit also refactors in order to provide a
working fix for authentication.
The original intent of the `ledger` facade was to describe a general
purpose "ledger" command abstraction; not specifically the `ledger`
software (predecessor to hledger) nor the `ledger` hardware device.
As `hledger` is currently the exclusive internal "ledger" component,
adding `hledger` as an alias seems to be an appropriate-enough addition
that won't break existing abstraction/functionality.
- Fetch from all available price APIs within a single command
* allows passing multiple CLI APIs to `api` argument
* will now fetch from all available APIs if `api` is not given
* will warn if API config block is missing or die of no APIs are found
- Related refactoring
