Results 340 comments of Dan Gould

@benalleng please pick up from #405 if you're up for it. Lots of small PRs will let us crank forward progress on this one. I'm not tied to getting #405...

There are a few remaining `receiver` Error variants that can probably be handled as `ReplyableError` depending on the receiver's preferences. ## `SelectionError` SelectionError arises from receiver coin selection failing, when...

- [ ] When we make error variants public, remember to reflect the change in tests by matching variants instead of messages.

From Tobin @ rust-payjoin > Hey mate, in rust-bitcoin we took the stance that we basically can't commit to any error types for the initial 1.0 releases so we used...

This kind of multiparty API change needs to be experimented with before payjoin-1.0 so we know that our stable API can accommodate it. It would be crappy to stabilize the...

It wouldn't kill us if this were saved for a later release than 1.0

> e -> HandlerError { e.into() } is probably also me being a rust n00b You're looking for `.map_err(HandlerError::from)?`

If I'm in this `nix develop` shell is this supposed to supply bitcoind for tests by default? I'm getting this error for every integration test that requires bitcoind when I...