From 8cfbf1f0a20e77c7c5716ed0ab145eef810aadaf Mon Sep 17 00:00:00 2001
From: Steve Myers <steve@notmandatory.org>
Date: Wed, 16 Dec 2020 11:33:34 -0800
Subject: [PATCH] [docs] Add more docs to 'types.rs'

---
 src/lib.rs   |  1 -
 src/types.rs | 13 ++++++++++++-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/lib.rs b/src/lib.rs
index d626b8a2..3991aed6 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -258,7 +258,6 @@ pub mod descriptor;
 mod doctest;
 pub mod keys;
 pub(crate) mod psbt;
-#[allow(missing_docs)] // TODO add missing docs and remove this allow
 pub(crate) mod types;
 pub mod wallet;
 
diff --git a/src/types.rs b/src/types.rs
index 07d344c5..4a60e2d9 100644
--- a/src/types.rs
+++ b/src/types.rs
@@ -29,7 +29,7 @@ use bitcoin::hash_types::Txid;
 
 use serde::{Deserialize, Serialize};
 
-/// Types of script
+/// Types of keychains
 #[derive(Serialize, Deserialize, Debug, Clone, Copy, PartialEq, Eq, Hash)]
 pub enum KeychainKind {
     /// External
@@ -39,6 +39,7 @@ pub enum KeychainKind {
 }
 
 impl KeychainKind {
+    /// Return [`KeychainKind`] as a byte
     pub fn as_byte(&self) -> u8 {
         match self {
             KeychainKind::External => b'e',
@@ -92,19 +93,29 @@ impl std::default::Default for FeeRate {
 /// A wallet unspent output
 #[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq)]
 pub struct UTXO {
+    /// Reference to a transaction output
     pub outpoint: OutPoint,
+    /// Transaction output
     pub txout: TxOut,
+    /// Type of keychain
     pub keychain: KeychainKind,
 }
 
 /// A wallet transaction
 #[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, Default)]
 pub struct TransactionDetails {
+    /// Optional transaction
     pub transaction: Option<Transaction>,
+    /// Transaction id
     pub txid: Txid,
+    /// Timestamp
     pub timestamp: u64,
+    /// Received value (sats)
     pub received: u64,
+    /// Sent value (sats)
     pub sent: u64,
+    /// Fee value (sats)
     pub fees: u64,
+    /// Confirmed in block height, `None` means unconfirmed
     pub height: Option<u32>,
 }