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 blocking option to payinvoice API #1751

Merged
merged 1 commit into from
Apr 2, 2021
Merged

Conversation

t-bast
Copy link
Member

@t-bast t-bast commented Apr 1, 2021

Add an option to block until the payment completes to the payinvoice API.
This can be simpler to use for payments that complete quickly than polling the inefficient getsentinfo API.

@t-bast t-bast requested a review from sstone April 1, 2021 14:02
@codecov-io
Copy link

Codecov Report

Merging #1751 (99e62d3) into master (936f36b) will decrease coverage by 0.08%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##           master    #1751      +/-   ##
==========================================
- Coverage   87.27%   87.18%   -0.09%     
==========================================
  Files         154      154              
  Lines       11989    12006      +17     
  Branches      508      522      +14     
==========================================
+ Hits        10463    10468       +5     
- Misses       1526     1538      +12     
Impacted Files Coverage Δ
...r-core/src/main/scala/fr/acinq/eclair/Eclair.scala 49.01% <0.00%> (-5.73%) ⬇️
...r/acinq/eclair/payment/send/PaymentInitiator.scala 96.66% <100.00%> (+0.03%) ⬆️
...nq/eclair/blockchain/electrum/ElectrumWallet.scala 80.25% <0.00%> (-0.26%) ⬇️
...c/main/scala/fr/acinq/eclair/channel/Channel.scala 86.52% <0.00%> (-0.17%) ⬇️
...main/scala/fr/acinq/eclair/router/Validation.scala 92.30% <0.00%> (+1.53%) ⬆️
...q/eclair/blockchain/electrum/ElectrumWatcher.scala 77.23% <0.00%> (+2.43%) ⬆️

Add an option to block until the payment completes to the payinvoice API.
This can be simpler to use for payments that complete quickly.
@t-bast t-bast merged commit 5729b28 into master Apr 2, 2021
@t-bast t-bast deleted the send-payment-blocking branch April 2, 2021 06:36
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.

3 participants