https://github.com/lightningnetwork/lnd/pull/2022/ will be the new standard and our https://github.com/lightningnetwork/lnd/pull/1820 won't get merged. Hence we should switch at some point. We'll definitely wait for 2022 to be in master tho.
Still not merged, moving to next milestone.
Merged.
I think we can tackle this one
I will take a look at this before our call to get an idea of how big a lift this change will be for us.
Here's a quick overview of what needs to happen for xud to support hold invoices:
AddHoldInvoice with the has. I'm thinking we can actually send the swap request packet and call AddHoldInvoice in parallel to eliminate an extra step of waiting.SubscribeSingleInvoice to be notified when someone tries to pay it.CancelInvoice and end the subscription if the swap fails or times out.SubscribeSingleInvoice, do our existing resolve hash routine and call SettleInvoice once we have the preimage.invoicesrpc flag to enable the hold invoices functionality.You can work with an official release now: https://github.com/lightningnetwork/lnd/releases/tag/v0.6-beta @sangaman 鈿★笍
Most helpful comment
Here's a quick overview of what needs to happen for xud to support hold invoices:
AddHoldInvoicewith the has. I'm thinking we can actually send the swap request packet and callAddHoldInvoicein parallel to eliminate an extra step of waiting.SubscribeSingleInvoiceto be notified when someone tries to pay it.CancelInvoiceand end the subscription if the swap fails or times out.SubscribeSingleInvoice, do our existing resolve hash routine and callSettleInvoiceonce we have the preimage.invoicesrpcflag to enable the hold invoices functionality.