use zcash_protocol's networkType and remove unused variable

Author: pacu

Cargo.lock

@@ -11,6 +11,7 @@ edition = "2021"
 serde.workspace = true
 zcash_primitives.workspace = true
 zcash_address.workspace = true
+zcash_protocol.workspace = true
 thiserror.workspace = true
 wasm-bindgen.workspace = true
 js-sys.workspace = true

crates/webzjs-common/Cargo.toml

@@ -4,7 +4,7 @@
 use crate::error::Error;
 use serde::{Deserialize, Serialize};
 use std::str::FromStr;
-use zcash_primitives::consensus::{self, Parameters};
+use zcash_protocol::consensus::{self, Parameters};
 
 /// Enum representing the network type
 /// This is used instead of the `consensus::Network` enum so we can derive
@@ -29,10 +29,10 @@ impl FromStr for Network {
 }
 
 impl Parameters for Network {
-    fn network_type(&self) -> zcash_address::Network {
+    fn network_type(&self) -> zcash_protocol::consensus::NetworkType {
         match self {
-            Network::MainNetwork => zcash_address::Network::Main,
-            Network::TestNetwork => zcash_address::Network::Test,
+            Network::MainNetwork => zcash_protocol::consensus::NetworkType::Main,
+            Network::TestNetwork => zcash_protocol::consensus::NetworkType::Test,
         }
     }
 

crates/webzjs-common/src/network.rs

@@ -131,7 +131,7 @@ pub async fn pczt_sign_inner(
         .finish();
     let mut signer = Signer::new(pczt).unwrap();
     //.map_err(|e| anyhow!("Failed to initialize Signer: {:?}", e))?;
-    for (account_index, spends) in keys {
+    for (_, spends) in keys {
         // let usk = UnifiedSpendingKey::from_seed(&params, seed, account_index)?;
         for keyref in spends {
             match keyref {