Use 'account_balance' instead of the legacy 'account_balance_dfx'. (#4173)

Author: vincent-dfinity

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 # UNRELEASED
 
+### chore: use `account_balance` instead of the legacy `account_balance_dfx`
+
+Use the `account_balance` rather than the legacy `account_balance_dfx` on the ICP ledger.
+
 ### feat: Extend `dfx ledger transfer` and `dfx ledger balance` to support ICRC-1 standard
 
 Extend `dfx ledger transfer` and `dfx ledger balance` to support [ICRC-1 standard](https://github.com/dfinity/ICRC-1/tree/main/standards/ICRC-1).

src/dfx/src/lib/ledger_types/mod.rs

@@ -121,7 +121,7 @@ pub struct Memo(pub u64);
 
 #[derive(CandidType)]
 pub struct AccountBalanceArgs {
-    pub account: String,
+    pub account: AccountIdBlob,
 }
 
 #[derive(CandidType)]

src/dfx/src/lib/operations/ledger.rs

@@ -36,7 +36,7 @@ use icrc_ledger_types::icrc2::transfer_from::TransferFromError;
 use slog::{info, Logger};
 use std::time::{SystemTime, UNIX_EPOCH};
 
-const ACCOUNT_BALANCE_METHOD: &str = "account_balance_dfx";
+const ACCOUNT_BALANCE_METHOD: &str = "account_balance";
 const TRANSFER_METHOD: &str = "transfer";
 
 pub async fn balance(
@@ -52,7 +52,7 @@ pub async fn balance(
     let (result,) = canister
         .query(ACCOUNT_BALANCE_METHOD)
         .with_arg(AccountBalanceArgs {
-            account: acct.to_string(),
+            account: acct.to_address(),
         })
         .build()
         .call()