gui(spend): display warnings for draft PSBTs

Author: jp1ac4

gui/src/app/message.rs

@@ -29,7 +29,7 @@ pub enum Message {
     Coins(Result<Vec<Coin>, Error>),
     Labels(Result<HashMap<String, String>, Error>),
     SpendTxs(Result<Vec<SpendTx>, Error>),
-    Psbt(Result<Psbt, Error>),
+    Psbt(Result<(Psbt, Vec<String>), Error>),
     Recovery(Result<SpendTx, Error>),
     Signed(Fingerprint, Result<Psbt, Error>),
     WalletRegistered(Result<Fingerprint, Error>),

gui/src/app/state/spend/step.rs

@@ -32,7 +32,7 @@ pub struct TransactionDraft {
     network: Network,
     inputs: Vec<Coin>,
     recipients: Vec<Recipient>,
-    generated: Option<Psbt>,
+    generated: Option<(Psbt, Vec<String>)>,
     batch_label: Option<String>,
     labels: HashMap<String, String>,
 }
@@ -83,7 +83,7 @@ pub struct DefineSpend {
     batch_label: form::Value<String>,
     amount_left_to_select: Option<Amount>,
     feerate: form::Value<String>,
-    generated: Option<Psbt>,
+    generated: Option<(Psbt, Vec<String>)>,
     warning: Option<Error>,
 }
 
@@ -384,7 +384,9 @@ impl Step for DefineSpend {
                                     .create_spend_tx(&inputs, &outputs, feerate_vb)
                                     .map_err(|e| e.into())
                                     .and_then(|res| match res {
-                                        CreateSpendResult::Success { psbt, .. } => Ok(psbt),
+                                        CreateSpendResult::Success { psbt, warnings } => {
+                                            Ok((psbt, warnings))
+                                        }
                                         CreateSpendResult::InsufficientFunds { missing } => {
                                             Err(SpendCreationError::CoinSelection(
                                                 liana::spend::InsufficientFunds { missing },
@@ -438,7 +440,7 @@ impl Step for DefineSpend {
             .filter_map(|(coin, selected)| if *selected { Some(coin) } else { None })
             .cloned()
             .collect();
-        if let Some(psbt) = &self.generated {
+        if let Some((psbt, _)) = &self.generated {
             draft.labels = self.coins_labels.clone();
             for (i, output) in psbt.unsigned_tx.output.iter().enumerate() {
                 if let Some(label) = self
@@ -583,7 +585,7 @@ impl Recipient {
 
 pub struct SaveSpend {
     wallet: Arc<Wallet>,
-    spend: Option<psbt::PsbtState>,
+    spend: Option<(psbt::PsbtState, Vec<String>)>,
     curve: secp256k1::Secp256k1<secp256k1::VerifyOnly>,
 }
 
@@ -599,7 +601,7 @@ impl SaveSpend {
 
 impl Step for SaveSpend {
     fn load(&mut self, draft: &TransactionDraft) {
-        let psbt = draft.generated.clone().unwrap();
+        let (psbt, warnings) = draft.generated.clone().unwrap();
         let mut tx = SpendTx::new(
             None,
             psbt,
@@ -623,12 +625,15 @@ impl Step for SaveSpend {
             }
         }
 
-        self.spend = Some(psbt::PsbtState::new(self.wallet.clone(), tx, false));
+        self.spend = Some((
+            psbt::PsbtState::new(self.wallet.clone(), tx, false),
+            warnings,
+        ));
     }
 
     fn subscription(&self) -> Subscription<Message> {
-        if let Some(spend) = &self.spend {
-            spend.subscription()
+        if let Some((psbt_state, _)) = &self.spend {
+            psbt_state.subscription()
         } else {
             Subscription::none()
         }
@@ -640,26 +645,27 @@ impl Step for SaveSpend {
         cache: &Cache,
         message: Message,
     ) -> Command<Message> {
-        if let Some(spend) = &mut self.spend {
-            spend.update(daemon, cache, message)
+        if let Some((psbt_state, _)) = &mut self.spend {
+            psbt_state.update(daemon, cache, message)
         } else {
             Command::none()
         }
     }
 
     fn view<'a>(&'a self, cache: &'a Cache) -> Element<'a, view::Message> {
-        let spend = self.spend.as_ref().unwrap();
+        let (psbt_state, warnings) = self.spend.as_ref().unwrap();
         let content = view::spend::spend_view(
             cache,
-            &spend.tx,
-            spend.saved,
-            &spend.desc_policy,
-            &spend.wallet.keys_aliases,
-            spend.labels_edited.cache(),
+            &psbt_state.tx,
+            warnings,
+            psbt_state.saved,
+            &psbt_state.desc_policy,
+            &psbt_state.wallet.keys_aliases,
+            psbt_state.labels_edited.cache(),
             cache.network,
-            spend.warning.as_ref(),
+            psbt_state.warning.as_ref(),
         );
-        if let Some(action) = &spend.action {
+        if let Some(action) = &psbt_state.action {
             action.as_ref().view(content)
         } else {
             content

gui/src/app/view/spend/mod.rs

@@ -33,6 +33,7 @@ use crate::{
 pub fn spend_view<'a>(
     cache: &'a Cache,
     tx: &'a SpendTx,
+    spend_warnings: &'a Vec<String>,
     saved: bool,
     desc_info: &'a LianaPolicy,
     key_aliases: &'a HashMap<Fingerprint, String>,
@@ -48,6 +49,21 @@ pub fn spend_view<'a>(
             .spacing(20)
             .push(Container::new(h3("Send")).width(Length::Fill))
             .push(psbt::spend_header(tx, labels_editing))
+            .push_maybe(if spend_warnings.is_empty() || saved {
+                None
+            } else {
+                Some(spend_warnings.iter().fold(
+                    Column::new().padding(15).spacing(5),
+                    |col, warning| {
+                        col.push(
+                            Row::new()
+                                .spacing(5)
+                                .push(icon::warning_icon().style(color::ORANGE))
+                                .push(text(warning).style(color::ORANGE)),
+                        )
+                    },
+                ))
+            })
             .push(psbt::spend_overview_view(tx, desc_info, key_aliases))
             .push(
                 Column::new()