- From: Domenic Denicola <notifications@github.com>
- Date: Mon, 06 Mar 2017 13:14:52 -0800
- To: w3c/browser-payment-api <browser-payment-api@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Monday, 6 March 2017 21:15:24 UTC
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