Re: [w3c/browser-payment-api] fix: delegate total and error member checks to WebIDL (#425)

domenic approved this pull request.

LGTM with nits

> @@ -329,9 +328,10 @@
           </pre>
         </div>
         <p>
-          The <a>PaymentRequest</a> constructor MUST act as follows:
+          The <a>PaymentRequest(methodData, details, options)</a> constructor

ReSpec linking error introduced here

> @@ -1118,11 +1093,11 @@
           <p class="note">

A few lines above this there's a `<a>shippingOption</a>` that should be `<a data-lt="PaymentRequest.shippingOption">shippingOption</a>` I think. Preexisting problem maybe?

-- 
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/425#pullrequestreview-25364357

Received on Monday, 6 March 2017 21:15:24 UTC