-
Notifications
You must be signed in to change notification settings - Fork 144
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(taro-lnd-link-details): info tab for taro-lnd link #667
Conversation
Added basic taro-lnd peer link info. Added test for taro-lnd link. Updated LinkDetails tests for taro nodes.
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.
Reviewed. I didn't really have any ideas for the taro lnd link node, so I copied the Peer component.
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.
Overall this works as expected. Just had a few comments on the code and labels.
Thanks for adding the tests to keep the coverage up. 👌
Is there an easy way to move this over to draft? I decided to add the change TaroBackend ability to this pr. Or should that be a separate pr? |
There is a "Convert to draft" link under the Reviewers section of this page. Definitely create a separate PR. These two features are unrelated, and this PR is pretty much done. |
Okay, thanks for the guidance. |
Proper casing on the usePrefixedTranslation
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.
tACK 👌
Great job on the tests.
Description
Added basic taro-lnd peer link info.
Added test for taro-lnd link.
Updated LinkDetails tests for taro nodes.
Steps to Test