From 45f91216d5d0991d7aebde8d4c8bc99edaf5521a Mon Sep 17 00:00:00 2001 From: jp1ac4 <121959000+jp1ac4@users.noreply.github.com> Date: Thu, 14 Sep 2023 14:11:41 +0100 Subject: [PATCH] gui(installer): remove unused fields from final step --- gui/src/installer/mod.rs | 7 +++---- gui/src/installer/step/mod.rs | 27 +++++++-------------------- 2 files changed, 10 insertions(+), 24 deletions(-) diff --git a/gui/src/installer/mod.rs b/gui/src/installer/mod.rs index 128ac34c..04ff4e22 100644 --- a/gui/src/installer/mod.rs +++ b/gui/src/installer/mod.rs @@ -120,7 +120,6 @@ impl Installer { } pub fn update(&mut self, message: Message) -> Command { - let hot_signer_fingerprint = self.signer.lock().unwrap().fingerprint(); match message { Message::CreateWallet => { self.steps = vec![ @@ -132,7 +131,7 @@ impl Installer { SelectBitcoindTypeStep::new().into(), InternalBitcoindStep::new(&self.context.data_dir).into(), DefineBitcoind::new().into(), - Final::new(hot_signer_fingerprint).into(), + Final::new().into(), ]; self.next() } @@ -147,7 +146,7 @@ impl Installer { SelectBitcoindTypeStep::new().into(), InternalBitcoindStep::new(&self.context.data_dir).into(), DefineBitcoind::new().into(), - Final::new(hot_signer_fingerprint).into(), + Final::new().into(), ]; self.next() } @@ -160,7 +159,7 @@ impl Installer { SelectBitcoindTypeStep::new().into(), InternalBitcoindStep::new(&self.context.data_dir).into(), DefineBitcoind::new().into(), - Final::new(hot_signer_fingerprint).into(), + Final::new().into(), ]; self.next() } diff --git a/gui/src/installer/step/mod.rs b/gui/src/installer/step/mod.rs index cfa579a5..e6664a9d 100644 --- a/gui/src/installer/step/mod.rs +++ b/gui/src/installer/step/mod.rs @@ -16,7 +16,6 @@ pub use mnemonic::{BackupMnemonic, RecoverMnemonic}; use std::path::PathBuf; use iced::{Command, Subscription}; -use liana::miniscript::bitcoin::bip32::Fingerprint; use liana_ui::widget::*; @@ -66,40 +65,28 @@ pub struct Final { internal_bitcoind: Option, warning: Option, config_path: Option, - hot_signer_fingerprint: Fingerprint, - hot_signer_is_not_used: bool, } impl Final { - pub fn new(hot_signer_fingerprint: Fingerprint) -> Self { + pub fn new() -> Self { Self { internal_bitcoind: None, generating: false, warning: None, config_path: None, - hot_signer_fingerprint, - hot_signer_is_not_used: false, } } } +impl Default for Final { + fn default() -> Self { + Self::new() + } +} + impl Step for Final { fn load_context(&mut self, ctx: &Context) { self.internal_bitcoind = ctx.internal_bitcoind.clone(); - if let Some(signer) = &ctx.recovered_signer { - self.hot_signer_fingerprint = signer.fingerprint(); - self.hot_signer_is_not_used = false; - } else if ctx - .descriptor - .as_ref() - .unwrap() - .to_string() - .contains(&self.hot_signer_fingerprint.to_string()) - { - self.hot_signer_is_not_used = false; - } else { - self.hot_signer_is_not_used = true; - } } fn load(&self) -> Command { if !self.generating && self.config_path.is_none() {