rpc method removeprunedfunds should take an array of txids #29466

1. 
   RHavar commented at 4:21 pm on February 22, 2024: contributor

   Please describe the feature you’d like to see added.

   Currently removeprunedfunds only takes a single txid, however the underlying CWallet::RemoveTxs takes an array. I am trying to clean up some wallets, and each individual removeprunedfunds call is very slow. So it’d be a lot better to just be able to pass in all the transactions at once

   Is your feature related to a problem, if so please describe it.

   No response

   Describe the solution you’d like

   No response

   Describe any alternatives you’ve considered

   No response

   Please leave any additional context

   No response

2. RHavar added the label Feature on Feb 22, 2024
3. maflcko added the label Wallet on Feb 22, 2024
4. 
   furszy commented at 6:54 pm on February 22, 2024: member

   Could add the “Good first issue” tag. And tag me for review whenever the PR is open, happy to check it.
5. 
   drkhero commented at 5:12 pm on April 3, 2024: none

   @furszy @glozow Still waiting on this issue to be fixed. Ryan has been waiting for a while now. Thanks!
6. 
   maflcko commented at 5:24 pm on April 3, 2024: member

   @drkhero A fix is in #29468 , but it is waiting for review and testing. If you want to move it forward, you can help with review and testing.

Contributors
RHavar furszy drkhero maflcko

Labels
Feature Wallet