Re: [w3c/webpayments-payment-handler] Editorial changes for FPWD (#138)

dlongley commented on this pull request.



>            </p>
         </section>
         <section>
           <h2>
             <dfn>wallets</dfn> attribute
           </h2>
           <p>
-            This attribute allows payment handlers to group payment instruments
+            This attribute allows service workers to group payment instruments

To throw another wrench in here ... perhaps `PaymentManager` should become `PaymentInstrumentStore` or `PaymentInstruments` or similar if it's not "managing" anything?

Regarding the original point, I don't think "service workers" are the ones doing the grouping, developers are -- they're the ones with agency here. Perhaps avoid the problem all together with:

`This attribute allows related payment instruments to be logically grouped together.`

or

`This attribute is used to group related payment instruments.`

-- 
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-handler/pull/138#discussion_r112000207