Wallet should use transaction version 2 when creating transactions

[ValuedMammal]

"You create transactions with nVersion set to 1 if no CSV. Nowadays >85% of transactions on the network are version 2, i think it would make you stick out. It's likely the <15% of version 1 transactions are created by old wallet. It's likely that BDK would be used with newer output types. If these two assumptions hold, it would make it possible to identify a BDK transaction with near certainty. Version 2 is also necessary if you want to eventually implement BIP 326 nSequence-based anti-fee-sniping."

[Freedisch]

This sounds like a good issue for me to start with, I will take a look at

[Gudnessuche]

Thoughts on adding V3 transaction types as well while at it?

[benalleng]

@Freedisch Ok if I pick this one up? and @Gudnessuche there is already support for any version above 0 including something non-standard like version 3.