From 8ea584e01fbb26f142887f5d74735db1d8db9bf2 Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Thu, 21 Jan 2021 22:21:34 +0000 Subject: [PATCH] Update bigtable confirm to use confirmation_status (#14750) (#14754) (cherry picked from commit ca95302038b70689fe740e2b2eca44a98fcfceac) Co-authored-by: Tyera Eulberg --- ledger-tool/src/bigtable.rs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/ledger-tool/src/bigtable.rs b/ledger-tool/src/bigtable.rs index 0faa6a60d4..83be98cb5a 100644 --- a/ledger-tool/src/bigtable.rs +++ b/ledger-tool/src/bigtable.rs @@ -106,14 +106,15 @@ async fn confirm(signature: &Signature, verbose: bool) -> Result<(), Box println!("Confirmed transaction details not available"), - Err(err) => println!("Unable to get confirmed transaction details: {}", err), + Ok(None) => println!("Finalized transaction details not available"), + Err(err) => println!("Unable to get finalized transaction details: {}", err), } println!(); } - match transaction_status.status { - Ok(_) => println!("Confirmed"), - Err(err) => println!("Transaction failed: {}", err), + if let Some(err) = &transaction_status.err { + println!("Transaction failed: {}", err); + } else { + println!("{:?}", transaction_status.confirmation_status()); } Ok(()) }