-
Notifications
You must be signed in to change notification settings - Fork 103
RC feedback fixes #61
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
Conversation
b825027
to
863376a
Compare
ad4a67b
to
427994f
Compare
5ac24ec
to
abe9249
Compare
603babf
to
f2401db
Compare
This PR is ready for review. There are still a couple outstanding tweaks that are still waiting for approval before adding. I'd prefer to get all the completed fixes merged instead of waiting on the rest. |
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, great fixes! Tested on top of the lnd v0.10.2-beta.rc1
branch, going to build new RC3 binaries now.
This PR will be an accumulation of small fixes and improvements found during internal testing.
yarn
command and updated screenshot to readmeIn Fee %
column Add alias and fees to channel list #55 (comment)Pending Confirmation