ord icon indicating copy to clipboard operation
ord copied to clipboard

`ord wallet cardinals` lists runic outputs too

Open gmart7t2 opened this issue 2 months ago • 1 comments

... but it shouldn't.

diff --git a/src/subcommand/wallet/cardinals.rs b/src/subcommand/wallet/cardinals.rs
index 1da96c4b..eb24b6e3 100644
--- a/src/subcommand/wallet/cardinals.rs
+++ b/src/subcommand/wallet/cardinals.rs
@@ -18,7 +18,7 @@ pub(crate) fn run(wallet: Wallet) -> SubcommandResult {
   let cardinal_utxos = unspent_outputs
     .iter()
     .filter_map(|(output, txout)| {
-      if inscribed_utxos.contains(output) {
+      if inscribed_utxos.contains(output) || !wallet.get_runes_balances_for_output(output).unwrap().is_empty() {
         None
       } else {
         Some(CardinalUtxo {

gmart7t2 avatar Apr 28 '24 10:04 gmart7t2

it would be great if, accompanying this fix an ord wallet runics is added to retrieve runic outputs. Preferably returned with its associated rune id, output, rune token balance, and runic balance.

razr22 avatar Apr 28 '24 17:04 razr22

Should be fixed now.

Added an issue for ord wallet runics: #3729

raphjaph avatar May 06 '24 11:05 raphjaph