FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Query listpays by label by ErikDeSmedt · Pull Request #9441 · ElementsProject/lightning · GitHub

Query listpays by label - #9441

Open
ErikDeSmedt wants to merge 2 commits into
ElementsProject:masterfrom
ErikDeSmedt:query-listpays-by-label
Open

Query listpays by label#9441
ErikDeSmedt wants to merge 2 commits into
ElementsProject:masterfrom
ErikDeSmedt:query-listpays-by-label

Conversation

ErikDeSmedt commented Aug 19, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

I want to query the status of a payment by

lightning-cli listpays label=my-label

Checklist

Before submitting the PR, ensure the following tasks are completed. If an item is not applicable to your PR, please mark it as checked:

  • The changelog has been updated in the relevant commit(s) according to the guidelines.
  • Tests have been added or modified to reflect the changes.
  • Documentation has been reviewed and updated as needed.
  • Related issues have been listed and linked, including any that this PR closes.
  • Important All PRs must consider how to reverse any persistent changes for tools/lightning-downgrade (Confirmed by Endothermic dev)

I have no clue what tools/lightning-downgrade is and need help from a maintainer if this is relevant for this PR.

ErikDeSmedt and others added 2 commits August 19, 2026 11:52
`listsendpays` accepts an optional `label` parameter and queries
payments by label; `listpays` forwards it. This matches `listinvoices`
which already supports lookup by label.

Changelog-Added: JSON-RPC: `listpays` and `listsendpays` can now be filtered by `label`.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Run `msggen generate` and protoc to pick up the new `label` request
field added to the listpays and listsendpays schemas.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
ErikDeSmedt force-pushed the query-listpays-by-label branch from 35f9cce to f8b9fc4 Compare August 19, 2026 10:01

Copy link
Copy Markdown
Collaborator

This doesn't touch the database, so you should be all set in the event of a downgrade.

Copy link
Copy Markdown
Contributor Author

I don't have a device running Mac OS and can't debug or run the tests.
Is it possible this failure was a flake.

Would a maintainer be willing to rerun the pipeline and see if t can become green?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL