New msgtx generation

ISTM this can only return an error when we generate the PkScript for the OutputRequest, but surely we could've detected that error early on instead of waiting until the end of the withdrawal process. That would also allow toMsgTx() to not return an error, but it's probably best to get this in first and I'll fix that on another branch