RPC: fix sendall docs #26010

1. 
   ajtowns commented at 8:45 AM on September 5, 2022: contributor

   Updates the documentation for the "inputs" entry in the "options" parameter of the sendall RPC to match the documentation for createrawtransaction.

2. 
   ajtowns commented at 8:45 AM on September 5, 2022: contributor

   cc @Xekyo @Sjors perhaps

3. in src/wallet/rpc/spend.cpp:1267 in 3b3409c838 outdated

   1265 | @@ -1266,9 +1266,13 @@ RPCHelpMan sendall()
   1266 |                                                "e.g. with 'importpubkey' or 'importmulti' with the 'pubkeys' or 'desc' field."},
   1267 |                          {"inputs", RPCArg::Type::ARR, RPCArg::Default{UniValue::VARR}, "Use exactly the specified inputs to build the transaction. Specifying inputs is incompatible with send_max. A JSON array of JSON objects",
   

   ---

   ---

   

   MarcoFalke commented at 9:17 AM on September 5, 2022:

                           {"inputs", RPCArg::Type::ARR, RPCArg::Default{UniValue::VARR}, "Use exactly the specified inputs to build the transaction. Specifying inputs is incompatible with send_max.",
   

   ---

   

   ajtowns commented at 1:25 PM on September 5, 2022:

   Done

4. MarcoFalke added this to the milestone 24.0 on Sep 5, 2022
5. DrahtBot added the label RPC/REST/ZMQ on Sep 5, 2022
6. glozow requested review from murchandamus on Sep 5, 2022
7. RPC: fix sendall docs

   Updates the documentation for the "inputs" entry in the "options"
   parameter of the sendall RPC to match the documentation for
   createrawtransaction.

   5182940996
8. ajtowns force-pushed on Sep 5, 2022
9. 
   ajtowns commented at 1:27 PM on September 5, 2022: contributor

   Current help text:

          "inputs": [                  (json array, optional, default=[]) Use exactly the specified inputs to build the transaction. Specifying inputs is incompatible with send_max. A JSON array of JSON objects
            "txid",                    (string, required) The transaction id
            vout,                      (numeric, required) The output number
            sequence,                  (numeric, required) The sequence number
            ...
          ],
   

   New help text:

          "inputs": [                  (json array, optional, default=[]) Use exactly the specified inputs to build the transaction. Specifying inputs is incompatible with send_max.
            {                          (json object)
              "txid": "hex",           (string, required) The transaction id
              "vout": n,               (numeric, required) The output number
              "sequence": n,           (numeric, optional, default=depends on the value of the 'replaceable' and 'locktime' arguments) The sequence number
            },
   

10. 
    achow101 commented at 9:40 PM on September 6, 2022: member

    ACK 51829409967dcfd039249506ecd2c58fb9a74b2f

11. 
    murchandamus commented at 9:53 PM on September 6, 2022: contributor

    ACK 51829409967dcfd039249506ecd2c58fb9a74b2f

12. achow101 merged this on Sep 6, 2022
13. achow101 closed this on Sep 6, 2022

    ---

14. sidhujag referenced this in commit 931b849c32 on Sep 7, 2022
15. bitcoin locked this on Sep 7, 2023

Contributors

ajtownsMarcoFalkeachow101murchandamus

Review Requested
murchandamus

Milestone
24.0