-
Notifications
You must be signed in to change notification settings - Fork 114
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. |
guggero
left a comment
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.
yarncommand and updated screenshot to readmeIn Fee %column Add alias and fees to channel list #55 (comment)Pending Confirmation