Re: [w3c/webpayments-payment-apps-api] Issue marker cleanup (#133)

adrianhopebailie approved this pull request.



> @@ -319,6 +319,9 @@
           readonly attribute PaymentManager paymentManager;
         };
       </pre>
+
+       <p class="issue" title="Issue 127 - Do we need Payment Manager and Payment Handler?"><a href="https://github.com/w3c/webpayments-payment-apps-api/issues/127">Issue 127</a>: The distinction is unclear. A proposal is to remove the phrase Payment Manager from the spec, renaming the "PaymentManager" interface the "PaymentHandler" interface.</p>
+

Suggest the issue can be closed. Can remove when done.

> @@ -1310,7 +1329,7 @@
         }));
       });
       </pre>
-         <p class="issue"><a href="https://github.com/w3c/webpayments-payment-apps-api/issues/128">Issue 128</a>: The Web Payments Working Group plans to revisit these two examples.</p>
+         <p class="issue" title="Issue 128 - Revisit examples"><a href="https://github.com/w3c/webpayments-payment-apps-api/issues/128">Issue 128</a>: The Web Payments Working Group plans to revisit these two examples.</p>

We should provide examples before FPWD or take the existing examples out. Most developers will jump directly to the examples and not see the issue marker.

-- 
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/webpayments-payment-apps-api/pull/133#pullrequestreview-32848968

Received on Friday, 14 April 2017 15:48:06 UTC