Re: [w3c/browser-payment-api] BREAKING CHANGE: change `supportedMethods` to DOMString (#551)

zcorpan commented on this pull request.



>            PaymentItem total;
           sequence<PaymentItem> additionalDisplayItems;
           object data;
         };
       </pre>
       <p>
         The <a>PaymentDetailsModifier</a> dictionary provides details that
-        modify the <a>PaymentDetailsBase</a> based on <a>payment method
-        identifier</a>. It contains the following fields:
+        modify the <a>PaymentDetailsBase</a> based on a <a>payment method
+        identifier</a>. It contains the following members:
       </p>
       <dl>
         <dt>
           <dfn>supportedMethods</dfn>

You could use a union `(DOMString or sequence<DOMString>)` and throw if the sequence doesn't have exactly 1 item. Then an array with multiple items can't "accidentally" (maybe intentionally, if the payment provider is a bit evil) end up being accepted as a supported payment identifier.

-- 
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/551#discussion_r122913358