- Pre-build transpiled source code (babel/stylus) and remove the devDependencies build tools from the final image.
- Allow to optionally keep devDependencies by setting the build arg TESTRUNNER.
- Shuffle around to optimize for minimal cache invalidation and fast build times during development.
Instead of tailing the log file for the init message, wait for the `.cookie` file and use `-rpcwait`.
This results in faster startup times and is less prone to breaking with new bitcoind releases.
- Use the new `status` argument for `delinvoice`, remove our manual status verification.
- Use the new `paid_at` field instead of using our local time when processing payments,
rename our `completed_at` to match (but still expose `completed_at` for now).
- Read from the `expires_at` field (renamed from `expiry_time`, now matches our local field name).
- test: `getpeers` -> `listpeers`
- test: Remove use of addfunds.
- test: Set --allow-deprecated-apis=false by default, overridable with ALLOW_DEPRECATED.
- Remove the `completed` column and determine completion status based on the presence of `pay_index`,
but still expose `completed` for now.
Not entirely related to c-lightning's API changes, but tightly coupled with the other changes.
Refs: https://github.com/ElementsProject/lightning/pull/623
Three possible states: `unpaid`, `paid` and `expired`.
The `completed` field is now deprecated as a public field
and will soon be hidden from the public APIs.
However, `completed` will continue to be used for internal storage,
with `status` as a virtual field determined based on `completed`, `expiry_time`
and the current time serving the request.
Nadav Ivgi