Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dni
Copy link
Member

@dni dni commented Feb 28, 2024

boolean if route hints for private channels will be included from LndRest inside bolt11 invoice.

note: fundingsources do not have more than text inputs yet when rendered

@codecov
Copy link

codecov bot commented Feb 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.49%. Comparing base (c8818f5) to head (ea3b654).

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2304      +/-   ##
==========================================
+ Coverage   58.47%   58.49%   +0.01%     
==========================================
  Files          61       61              
  Lines        9265     9266       +1     
==========================================
+ Hits         5418     5420       +2     
+ Misses       3847     3846       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@motorina0
Copy link
Collaborator

note: fundingsources do not have more than text inputs yet when rendered

you mean the dynamic component does not support bool/checkbox?

@motorina0 motorina0 self-requested a review March 11, 2024 13:58
@motorina0 motorina0 added this to the 0.12.3 milestone Mar 11, 2024
@dni dni requested a review from prusnak March 12, 2024 06:43
@dni dni requested review from arcbtc and talvasconcelos March 21, 2024 10:10
@dni
Copy link
Member Author

dni commented Mar 21, 2024

note: fundingsources do not have more than text inputs yet when rendered

you mean the dynamic component does not support bool/checkbox?

yes

dni added 2 commits March 22, 2024 12:36
boolean if route hints for private channels will be included from `LndRest` inside bolt11 invoice.

note: fundingsources do not have  more than text inputs yet when rendered
@dni dni merged commit 5b022e2 into dev Mar 22, 2024
@dni dni deleted the lndrest-hints branch March 22, 2024 11:45
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.

4 participants