Re: [w3c/browser-payment-api] Merged PaymentRequest params and tweaked to address some issues (#133)

>        <pre class="idl">
> -dictionary PaymentDetails {
> -  sequence&lt;PaymentItem&gt; items;
> -  sequence&lt;ShippingOption&gt; shippingOptions;
> +dictionary PaymentMethod {
> +  required sequence&lt;DOMString&gt; identifiers;
> +  object data;
>  };

I'm very wary of catch-all containers such as "data". I'd rather we figure out how we do extensibility properly before punting to "generic data buckets". So, -1 to this line. The rest of the PR is looking good - still reviewing it.

---
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/browser-payment-api/pull/133/files/38f22264a20bbe3dcaaa3d47036d9c08c07237fc#r59144519

Received on Monday, 11 April 2016 00:07:42 UTC