Sachin Meier
                                            Sachin Meier
                                        
                                    This is my first time reviewing an LND PR, but I can utACK it for now. If I get time i can test it soon(tm).
I am interested in picking this up as a first issue to contribute with. From my initial read of the code mentioned, it seems like the best way to handle...
This makes sense. If the payment fails, the messages will be exposed as errors, but in the case that it does not, I'm going to drop these messages down to...
yea, i'm going to leave it as a Warning. And thank you!
I'm also going to add logic for properly logging the other error types emitted at this same stage. It seems short-sighted to add only the maxBandwidth one.
@C-Otto I opened a [PR](https://github.com/lightningnetwork/lnd/pull/6756/) to address this. It is very similar to your patch. However, upon thinking through some of the logic, I realized that a channel can be...
I am interested in picking this one up. It seems to me that if the flag is not set, the value should not be changed, rather than setting a default...
I opened a PR to fix exactly this. It is very small. Any review is greatly appreciated. https://github.com/lightningnetwork/lnd/pull/6762
Updated commit message, thanks!
> @yyforyongyu: review reminder @SachinMeier, remember to re-request review from reviewers when ready Thank you. @C-Otto has an alternative MR to switch to v2 of the cli. @Roasbeef didn't seem...