Lloyd Fournier
Lloyd Fournier
I think you're right. I would expect patch to somehow relax this rule or provide a way of more powerfully overriding.
I think this works @thomaseizinger? if so I guess this can be closed.
Hey @q-src. I noticed that you didn't implement [BIP-137](https://github.com/bitcoin/bips/blob/master/bip-0137.mediawiki) or [BIP-322](https://github.com/bitcoin/bips/blob/master/bip-0322.mediawiki) are they not applicable to what you want to do? It's not clear this functionality needs a trait around...
@q-src TBH miniscript doesn't sound like the right place either. It might make sense to have a BIP322 implementation in `bdk` if it is supported somehow by the existing components...
> Note that the main goal for this task is not really to add more descriptors to the `Wallet` structure, but mainly to find a way to group multiple wallets...
Yep this is still a problem -- that line of code is still there.
what's wrong with ``` assert_eq!(wallet.sign(&mut psbt, options), Err(Error::Signer(SignerError::InputIndexOutOfRange))) ``` ?
Happy to mentor someone on this issue if anyone wants to have a shot. It looks like fun!
FWIW @csralvall I agree with removing Database entirely here. My approach to coin selection in `bdk_core` is to not have the UTXO type be fixed. i.e. apps can make their...
I suppose you'd have to use database or OS specific technology for this? Does SQLite have this feature?