-
Notifications
You must be signed in to change notification settings - Fork 149
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
feat:added LND connect URL for REST API #484
Conversation
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.
This looks good, I just requested a very minor change. Otherwise, great job!
values[l('invoiceUrl')] = encode({ host, cert, macaroon: invoiceMac }); | ||
values[l('readOnlyUrl')] = encode({ host, cert, macaroon: readonlyMac }); | ||
|
||
values[l('GRPCadminUrl')] = encode({ host, cert, macaroon: adminMac }); |
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.
Just a small nitpick, can you change these keys to camel case. ex: grpcAdminUrl
, restInvoiceUrl
, etc.
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.
ok I will do that
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.
LGTM
Codecov Report
@@ Coverage Diff @@
## master #484 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 113 113
Lines 3267 3271 +4
Branches 572 572
=========================================
+ Hits 3267 3271 +4
Continue to review full report at Codecov.
|
Closes #478
Description
added URL for REST API
Steps to Test
Screenshots