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

Include routing hints in parseinvoice API call response #1833

Merged
merged 1 commit into from
Jun 1, 2021

Conversation

sstone
Copy link
Member

@sstone sstone commented Jun 1, 2021

JSON objects returned by parseinvoice do not include routing hints, which can be misleading when decoding payment requests created by mobile wallets.

JSON objects returned by parseinvoice do not include routing hints which can be misleading when decoding payment requests created by mobile wallets.
@codecov-commenter
Copy link

Codecov Report

Merging #1833 (065a2e8) into master (af618bc) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1833      +/-   ##
==========================================
- Coverage   89.37%   89.36%   -0.01%     
==========================================
  Files         145      145              
  Lines       10980    10980              
  Branches      442      442              
==========================================
- Hits         9813     9812       -1     
- Misses       1167     1168       +1     
Impacted Files Coverage Δ
...c/main/scala/fr/acinq/eclair/channel/Channel.scala 86.19% <0.00%> (-0.09%) ⬇️

Copy link
Member

@pm47 pm47 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sstone sstone changed the title api/parseinvoice: serialise routing hints Include routing hints in parseinvoice API call response Jun 1, 2021
@sstone sstone merged commit dbecb28 into master Jun 1, 2021
@sstone sstone deleted the api-parseinvoice-fix branch June 1, 2021 14:06
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