-
Notifications
You must be signed in to change notification settings - Fork 108
Custom channels liquidity edge case: AddInvoice with existing route hints #1106
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
base: tapd-main-branch
Are you sure you want to change the base?
Custom channels liquidity edge case: AddInvoice with existing route hints #1106
Conversation
We can now use the LND_REPO, TAPROOT_ASSETS_REPO, TAPRPC_REPO, and LOOP_RPC build arguments to force a specific repo to be used so that commits referenced by LND_VERSION, TAPROOT_ASSETS_VERSION, TAPRPC_VERSION, and LOOP_VERSION don't have to exist in the default repository. If any of these build arguments are not defined, the build continues using the default repository for that module. NOTE: If these arguments ARE defined then the corresponding `_VERSION` argument MUST also be defined, otherwise the build continues using the default repository defined for that module.
…-itest Multi rfq send itest
6221457
to
396bed5
Compare
This PR didn't get merged on time when the tapd counterpart got merged: lightninglabs/taproot-assets#1627 Rebased on |
396bed5
to
2fe7d87
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, LGTM 🎉
I've increased the linter timeout in the tapd-main-branch
branch. If you rebase the failures should go away.
2fe7d87
to
c58b6d3
Compare
Rebased |
@ffranr: review reminder |
d59f943
to
4c6c117
Compare
Description
Adds a new edge case to the custom channels test suite, where we call the
taprpc.AddInvoice
with existing route hints, which include a valid accepted quote.Related tapd PR: lightninglabs/taproot-assets#1627