BIP 0070: Fix output use #1869

pull dancer1325 wants to merge 1 commits into bitcoin:master from dancer1325:patch-1 changing 1 files +1 −1

dancer1325 commented at 8:02 AM on June 13, 2025: none
description
- Output is used in PaymentDetails rather than PaymentRequest
how to review?
- https://github.com/bitcoin/bips/blob/master/bip-0070/paymentrequest.proto#L20
- https://github.com/bitcoin/bips/blob/master/bip-0070/paymentrequest.proto#L27-L33
BIP 0070: Fix output use a9177291fd
in bip-0070.mediawiki:58 in a9177291fd
```
  54 | @@ -55,7 +55,7 @@ The Protocol Buffers messages are defined in [[bip-0070/paymentrequest.proto|pay
  55 |  
  56 |  ===Output===
  57 |  
  58 | -Outputs are used in PaymentRequest messages to specify where a payment (or
```
murchandamus commented at 7:17 PM on June 16, 2025:

From what I understand, the PaymentDetails are wrapped inside PaymentRequest, so this seems fine to me.

jonatack commented at 9:26 PM on June 16, 2025:

Agree

dancer1325 commented at 12:37 PM on June 17, 2025:

From what I understand, the PaymentDetails are wrapped inside PaymentRequest, so this seems fine to me.

but not explicitly, thats' why I suggested that small change, more precise. But clear #1869 (comment)
jonatack commented at 9:43 PM on June 16, 2025: member

@dancer1325 Thank you for your diligence. This BIP is in final status since 2014 (8571f42a6), the BIP authors are unlikely to sign-off on further changes, and the current text looks fine, so closing here -- thanks!
jonatack closed this on Jun 16, 2025

Contributors

description