wallet: Add scan_utxo option to getbalances RPC #28930

pull BrandonOdiwuor wants to merge 1 commits into bitcoin:master from BrandonOdiwuor:checkbalancerpc changing 6 files +69 −4
  1. BrandonOdiwuor commented at 3:27 pm on November 23, 2023: contributor
    Fixes #28898
  2. DrahtBot commented at 3:27 pm on November 23, 2023: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

    Conflicts

    No conflicts as of last run.

  3. DrahtBot added the label Wallet on Nov 23, 2023
  4. maflcko commented at 3:41 pm on November 23, 2023: member

    Could add tests for:

    • Tx in mempool
    • Watchonly wallet

    ?

  5. DrahtBot added the label CI failed on Nov 23, 2023
  6. fjahr commented at 5:37 pm on November 23, 2023: contributor
    I think I would prefer the other option @maflcko suggested in the issue, i.e. adding a check flag to getbalances.
  7. wallet: Add scan_utxo option to getbalances RPC
    Add the option to getbalances by scanning the utxo set
    b210d3b134
  8. BrandonOdiwuor force-pushed on Nov 28, 2023
  9. BrandonOdiwuor renamed this:
    wallet: Add checkbalance RPC
    wallet: Add scan_utxo option to getbalances RPC
    on Nov 28, 2023
  10. DrahtBot removed the label CI failed on Nov 28, 2023
  11. luke-jr commented at 1:40 am on December 5, 2023: member
    getbalances seems like the wrong place for this?
  12. DrahtBot added the label CI failed on Jan 16, 2024

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-06-29 07:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me