Skip to content

Deprecate xpay-handle-pay#9254

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

Deprecate 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.

@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 3 times, most recently from 89f8387 to 72d3263 Compare July 1, 2026 13:00
Lagrang3 added 3 commits July 9, 2026 15:14
Changelog-None

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>
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>
- 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 72d3263 to 3660f6f Compare July 9, 2026 14:14
@Lagrang3 Lagrang3 requested a review from daywalker90 as a code owner July 9, 2026 14:14
@Lagrang3

Lagrang3 commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator Author
  • fixed errors in tests due to the deprecation,
  • rebased

@Lagrang3 Lagrang3 added the Status::Ready for Review The work has been completed and is now awaiting evaluation or approval. label Jul 9, 2026
@Lagrang3 Lagrang3 added this to the v26.09 milestone Jul 9, 2026
@Lagrang3 Lagrang3 changed the title Deprecated xpay-handle-pay Deprecate xpay-handle-pay Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status::Ready for Review The work has been completed and is now awaiting evaluation or approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant