Skip to content

Deprecated xpay-handle-pay#9254

Open
Lagrang3 wants to merge 3 commits into
ElementsProject:masterfrom
Lagrang3:remove-pay-backend
Open

Deprecated xpay-handle-pay#9254
Lagrang3 wants to merge 3 commits into
ElementsProject:masterfrom
Lagrang3:remove-pay-backend

Conversation

@Lagrang3

@Lagrang3 Lagrang3 commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

@rustyrussell deprecated pay in #9110, but he forgot to deprecate xpay-handle-pay.
This PR deprecates xpay-handle-pay for v26.09.

Changelog-None

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>
@Lagrang3 Lagrang3 force-pushed the remove-pay-backend branch from fa2c776 to fa8038a Compare July 1, 2026 07:31
@Lagrang3 Lagrang3 changed the title Remove pay backend Deprecated xpay-handle-pay Jul 1, 2026
@Lagrang3 Lagrang3 marked this pull request as ready for review July 1, 2026 07:34
@Lagrang3 Lagrang3 requested a review from cdecker as a code owner July 1, 2026 07:34
@Lagrang3 Lagrang3 requested a review from nGoline July 1, 2026 07:35
@Lagrang3 Lagrang3 force-pushed the remove-pay-backend branch from fa8038a to 8f95442 Compare July 1, 2026 07:36
Changelog-Deprecated: Config: option `xpay-handle-pay`, `pay` is deprecated and will be replaced by `xpay` in v27.03.

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>
@Lagrang3 Lagrang3 force-pushed the remove-pay-backend branch from 8f95442 to 89f8387 Compare July 1, 2026 10:23
- add missing deprecation annotation to "pay" command,
- fix tests that required allow-deprecated-apis=True in order to
continue using "pay",
- change "pay" to "xpay" in those tests that
are NOT testing "pay" directly but just want to make a payment,
- and in tests that could be relevant for both "xpay" and "pay", use
  both.

Changelog-None

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>
@Lagrang3 Lagrang3 force-pushed the remove-pay-backend branch from 89f8387 to 72d3263 Compare July 1, 2026 13:00
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.

1 participant