Updated listsinceblock rpc documentation #9396

pull accraze wants to merge 1 commits into bitcoin:master from accraze:docs-listsinceblock-rpc changing 1 files +1 −0
  1. accraze commented at 1:43 am on December 21, 2016: contributor

    This PR contains updates to the documentation for listsinceblock rpc.

    fixes #8758

  2. fanquake added the label RPC/REST/ZMQ on Dec 21, 2016
  3. MarcoFalke added the label Docs and Output on Dec 21, 2016
  4. updated listsinceblock rpc docs
    fixes #8758
    3f67972654
  5. in src/wallet/rpcwallet.cpp: in efba947828 outdated
    1636@@ -1637,7 +1637,7 @@ UniValue listsinceblock(const JSONRPCRequest& request)
    1637             "\nGet all transactions in blocks since block [blockhash], or all transactions if omitted\n"
    1638             "\nArguments:\n"
    1639             "1. \"blockhash\"   (string, optional) The block hash to list transactions since\n"
    1640-            "2. target-confirmations:    (numeric, optional) The confirmations required, must be 1 or more\n"
    1641+            "2. target-confirmations:    (numeric, optional) Return the hash of the block that has target-confirmations of confirmations, for use in a further call to listsinceblock.\n"
    


    paveljanik commented at 7:27 am on December 21, 2016:
    This is surely wrong. It should describe the meaning of the argument. And it does so in the master.
  6. accraze force-pushed on Dec 22, 2016
  7. accraze commented at 8:52 pm on December 22, 2016: contributor
    @paveljanik I removed the incorrect description
  8. instagibbs commented at 7:41 pm on December 30, 2016: member

    nit: maybe add “e.g., -5 implies 5 blocks ago”, in case someone reads it as “-5 blocks ago”(5 blocks in the future?)

    ACK https://github.com/bitcoin/bitcoin/pull/9396/commits/3f67972654748a2f24e4aa95b6f3df5d4574ed55

  9. laanwj merged this on Jan 4, 2017
  10. laanwj closed this on Jan 4, 2017

  11. laanwj referenced this in commit d65a13b773 on Jan 4, 2017
  12. codablock referenced this in commit a81b2750fc on Jan 18, 2018
  13. andvgal referenced this in commit 82473e1604 on Jan 6, 2019
  14. CryptoCentric referenced this in commit c3aebfc247 on Feb 26, 2019
  15. MarcoFalke locked this on Sep 8, 2021

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-11-25 21:12 UTC

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