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