- From: ianbjacobs <notifications@github.com>
- Date: Mon, 22 Jan 2018 14:45:59 -0800
- To: w3c/payment-request <payment-request@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Monday, 22 January 2018 22:46:40 UTC
ianbjacobs approved this pull request. I support the edit. Just a note about relation to payment handler API: the "modifiers" member of the PaymentRequestEvent comes from PaymentDetailsModifier. However, the algorithm [1] that describes how "modifiers" is populated (already) does not include displayItems. So I think no change is required to paymentHandler. [1] https://w3c.github.io/payment-handler/#dfn-modifiers-population-algorithm -- 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-request/pull/670#pullrequestreview-90654128
Received on Monday, 22 January 2018 22:46:40 UTC