Flatten tx errors

[kevinheavey]

This changes TransactionResult to be a std::result::Result, where the Ok() variant contains a TransactionMetadata, and the Err() variant contains both the TransactionMetadata and the TransactionError. This makes it more obvious to the user that send_transaction can return an error, and makes it easier to handle