Add wallet rescans to Properties

[kn0wmad]

In order to give user feedback without them going into the logs and discovering "benign" RPC errors

[ProofOfKeags]

what would this measure?

[kn0wmad]

%age of scan complete, similar to sync. @chrisguida suggested this could be a health check, although it would really only be relevant during a wallet scan. The rest of the time it's perhaps clutter

[chrisguida]

This could also go in properties

[ProofOfKeags]

What do you want to happen in if there is no wallet scan in progress? A green health check? No health check?

[chrisguida]

We could disable it if there's no scan in progress. Would probably want an option in the UI to hide it if it's disabled.

A better solution is probably to just put it in properties, rather than implement this as a health check.

[ProofOfKeags]

A better solution is probably to just put it in properties, rather than implement this as a health check.

+1

[Reckless-Satoshi]

Is there an action to start a wallet rescan or does it have to be triggered from CLI bitcoin-cli rescanblockchain?

I think an action like this could be created:

actions:
  # ...
  # reindex
  # delete-peers
  # ...
  rescan:
      name: "Rescan Blockchain"
      description: "Rescans the blockchain for Bitcoin Core wallet transactions. Useful when recovering old wallets on StartOS apps that depend on Bitcoin Core wallet functionalities."
      warning: The rescan will start from block zero and can take several hours to finish. 
      allowed-statuses:
        - running
      implementation:
        type: script  # something that triggers bitcoin-cli rescanblockchain
  # ...

Ideally, a parameter can be passed for the initial_blockheight or date to start the rescan (by default rescans from block 0). Extra cool if the rescan progress is shown in the UI as commented above.