add TODO about key_id in ledgertm signer

tendermint · Feb 20, 2019 · bdaa630 · bdaa630
1 parent 2dd4809
commit bdaa630
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/src/keyring/ed25519/ledgertm.rs b/src/keyring/ed25519/ledgertm.rs
@@ -30,6 +30,7 @@ pub fn init(
     }
     let provider = Box::new(Ed25519LedgerTmAppSigner::connect()?);
     let pk = provider.public_key()?;
+    // TODO: key_id shouldn't be a constant here (see LEDGER_TM_ID):
     let signer = Signer::new(LEDGER_TM_PROVIDER_LABEL, LEDGER_TM_ID.to_string(), provider);
     keyring.add(pk, signer)?;
     Ok(())