Oli

Results 502 comments of Oli
trafficstars

> But it is not compatible with the option to move to structured sql tables. This approach also isn't _blocking_ anything when it comes to moving to structured tables. On...

> Do you mean changing btcutil? That sounds like a pretty big behavioral change in how DecodeAddress works Yes, maybe it's too big of a change. But maybe it would...

> Looking at the code, this seems to only be possible with non-Segwit (non-bech32) addresses in the first place. Oops, I was wrong there. I read the `if chaincfg.IsBech32SegwitPrefix(prefix) {`...

You should never restore the `channel.db` file from a backup! You'll not only put your funds at risk but cause all sorts of problems. The watchtower not being able to...

We don't normally assign issues to external contributors. But if you want to "claim" something, you can always create a draft (!) pull request and link the issue in the...

> Did another pass, couldn't find the merge commit you mentioned Yeah, a rebase gets rid of such merge commits, so it's gone now :+1:

The new itest seems to fail at the moment.

Can you re-request review once the itests are fixed please?