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 LSPS1 client-side integration #418

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

tnull
Copy link
Collaborator

@tnull tnull commented Dec 4, 2024

We add the capability to act as an LSPS1 / bLIP-51 client.

@tnull tnull marked this pull request as draft December 4, 2024 14:49
@tnull tnull force-pushed the 2024-07-lsps1-integration branch from a160bf2 to d0eb977 Compare December 4, 2024 14:55
@tnull tnull force-pushed the 2024-07-lsps1-integration branch from d0eb977 to fe59fed Compare December 10, 2024 13:37
@tnull tnull force-pushed the 2024-07-lsps1-integration branch 2 times, most recently from 82fc329 to 9d47470 Compare January 10, 2025 12:09
@tnull tnull force-pushed the 2024-07-lsps1-integration branch from 9d47470 to 5e13aa5 Compare January 16, 2025 09:33
@tnull tnull marked this pull request as ready for review January 16, 2025 09:33
@tnull tnull added this to the 0.5 milestone Jan 16, 2025
tnull added 5 commits January 17, 2025 09:47
.. to align with the rest of the APIs where we usually go `node_id`,
`address`, etc.
We add support for LSPS1 liquidity sources. To this end we slightly
refactor our logic to first create a `LiquiditySourceBuilder` that then
can be used to `build()` the `LiquiditySource` with the configured
services.
We add the logic required to send `create_order` requests and check on
their status.
We add an `Lsps1Liquidity` API object, mirroring the approach we took
with the `payment` APIs.
@tnull tnull force-pushed the 2024-07-lsps1-integration branch from 5e13aa5 to 2780049 Compare January 17, 2025 08:47
@tnull
Copy link
Collaborator Author

tnull commented Jan 17, 2025

Rebased on main after #426 landed.

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