Refactor TransactionDetails to include confirmation_time

[notmandatory]

Description

Fixes #180

Notes to the reviewers

Checklists

All Submissions:

• I've signed all my commits
• I followed the contribution guidelines
• I ran cargo fmt and cargo clippy before committing

New Features:

• I've added tests for the new feature
• I've added docs for the new feature
• I've updated CHANGELOG.md

Bugfixes:

• This pull request breaks the existing API
• I've added tests to reproduce the issue which are now passing
• I'm linking the issue being fixed by this PR

[thunderbiscuit]

@notmandatory is this ready for review or do you still see it as draft? I'll take a look today if it's ready.

[notmandatory]

It's ready to review!

[thunderbiscuit]

Tested ACK. Small typo in the changelog but that's it. 🚀

[thunderbiscuit]

Rust-related question: why does confirmation_time here require to be cloned and the others don't?

impl From<&bdk::TransactionDetails> for TransactionDetails {
    fn from(x: &bdk::TransactionDetails) -> TransactionDetails {
        TransactionDetails {
            fee: x.fee,
            txid: x.txid.to_string(),
            received: x.received,
            sent: x.sent,
            confirmation_time: x.confirmation_time.clone(),
        }
    }
}

[notmandatory]

Rust-related question: why does confirmation_time here require to be cloned and the others don't?

This is because the other types have the Copy trait, meaning they are native types that can be copied into the TransactionDetails automatically. But confirmation_time is a structure that we created so it needs to be explicitly cloned from our reference to the bdk versionbdk::TransactionDetails.