- From: Rouslan Solomakhin <notifications@github.com>
- Date: Wed, 17 Oct 2018 09:52:37 -0700
- To: w3c/payment-handler <payment-handler@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Wednesday, 17 October 2018 16:52:59 UTC
rsolomakhin commented on this pull request. > + <dt> + <a data-lt= + "PaymentRequestEvent.instrumentKey">instrumentKey</a> + </dt> + <dd> + The <a>instrumentKey</a> of the selected + <a>PaymentInstrument</a>, or the empty string if none was + selected. + </dd> + <dt> + <a data-lt="PaymentRequestEvent.modifiers">modifiers</a> + </dt> + <dd> + The result of executing the <a>Modifiers Population + Algorithm</a>. + </dd> Could you move these fields from `canmakepayment` event to `paymentrequest` event, please? - total - paymentRequestId - instrumentKey -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/w3c/payment-handler/pull/327#pullrequestreview-165734289
Received on Wednesday, 17 October 2018 16:52:59 UTC