Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pagination on listInvoices #2474

Merged
merged 3 commits into from
Nov 14, 2022
Merged

Add pagination on listInvoices #2474

merged 3 commits into from
Nov 14, 2022

Conversation

rorp
Copy link
Contributor

@rorp rorp commented Nov 2, 2022

@codecov-commenter
Copy link

Codecov Report

Merging #2474 (e28c0c5) into master (3b12475) will increase coverage by 0.01%.
The diff coverage is 92.26%.

❗ Current head e28c0c5 differs from pull request most recent head 6527bc1. Consider uploading reports for the commit 6527bc1 to get more accurate results

@@            Coverage Diff             @@
##           master    #2474      +/-   ##
==========================================
+ Coverage   84.92%   84.93%   +0.01%     
==========================================
  Files         198      198              
  Lines       15783    15815      +32     
  Branches      637      682      +45     
==========================================
+ Hits        13404    13433      +29     
- Misses       2379     2382       +3     
Impacted Files Coverage Δ
...in/scala/fr/acinq/eclair/channel/ChannelData.scala 98.11% <ø> (ø)
.../main/scala/fr/acinq/eclair/db/DualDatabases.scala 11.27% <0.00%> (+0.32%) ⬆️
.../src/main/scala/fr/acinq/eclair/db/NetworkDb.scala 100.00% <ø> (ø)
...src/main/scala/fr/acinq/eclair/db/PaymentsDb.scala 84.61% <ø> (ø)
.../eclair/wire/protocol/LightningMessageCodecs.scala 100.00% <ø> (ø)
...in/scala/fr/acinq/eclair/channel/fsm/Channel.scala 86.27% <66.66%> (+0.11%) ⬆️
...r-core/src/main/scala/fr/acinq/eclair/Eclair.scala 52.10% <80.00%> (+1.56%) ⬆️
...main/scala/fr/acinq/eclair/router/Validation.scala 94.40% <87.23%> (-1.02%) ⬇️
.../scala/fr/acinq/eclair/message/OnionMessages.scala 81.81% <90.00%> (+5.45%) ⬆️
...src/main/scala/fr/acinq/eclair/router/Router.scala 94.20% <93.33%> (+0.02%) ⬆️
... and 26 more

Copy link
Member

@t-bast t-bast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing that @rorp!

@t-bast t-bast self-requested a review November 10, 2022 16:23
Copy link
Member

@t-bast t-bast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@t-bast t-bast merged commit 8f1af28 into ACINQ:master Nov 14, 2022
t-bast added a commit that referenced this pull request Nov 14, 2022
This is a follow-up for #2474: listing pending invoices should also benefit
from `skip` and `count` pagination.
t-bast added a commit that referenced this pull request Nov 14, 2022
The default values were generated once when the eclair node starts instead
of being recomputed for every request. Fixes #2475.

We also add pagination to the `listPendingInvoice` API as a follow-up for
#2474.
t-bast added a commit that referenced this pull request Dec 1, 2022
The release introduces a few API changes:

- channelbalances retrieves information about the balances of all local
  channels (#2196)
- channelbalances and usablebalances return a shortIds object instead
  of a single shortChannelId (#2323)
- stop stops eclair: please note that the recommended way of stopping
  eclair is simply to kill its process (#2233)
- rbfopen lets the initiator of a dual-funded channel RBF the funding
  transaction (#2275)
- listinvoices and listpendinginvoices now accept --count and --skip
  parameters to limit the number of retrieved items (#2474)
t-bast added a commit that referenced this pull request Jun 16, 2023
This release introduces a few API changes:

- `audit` now accepts `--count` and `--skip` parameters to limit the
  number of retrieved items (#2474, #2487)
- `sendtoroute` removes the `--trampolineNodes` argument and
  implicitly uses a single trampoline hop (#2480)
- `sendtoroute` now accept `--maxFeeMsat` to specify an upper bound
  of fees (#2626)
- `payinvoice` always returns the payment result when used with
  `--blocking`, even when using MPP (#2525)
- `node` returns high-level information about a remote node (#2568)
- `channel-created` is a new websocket event that is published when
  a channel's funding transaction has been broadcast (#2567)
- `channel-opened` websocket event was updated to contain the final
  `channel_id` and be published when a channel is ready to process
  payments (#2567)
- `getsentinfo` can now be used with `--offer` to list payments sent
  to a specific offer
- `listreceivedpayments` lists payments received by your node (#2607)
- `closedchannels` lists closed channels. It accepts `--count` and
  `--skip` parameters to limit the number of retrieved items as well
  (#2642)
- `cpfpbumpfees` can be used to unblock chains of unconfirmed
  transactions by creating a child transaction that pays a high fee
  (#1783)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add pagination on listInvoices
3 participants