-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Should list of accepted payment methods be strings or objects? #37
Comments
They should be objects for at least the following reasons:
|
I agree that I see no value in splitting the list of supported methods and the meta-data for each method into separate parameters. I also think the format proposed by the Web Payments CG is more flexible as it allows data such as the amount to be different for different payment methods. The following issue is related and provides an example of a request that is something of a combination of the two: https://github.com/WICG/paymentrequest/issues/41 |
+1 to objects for reasons stated by both @msporny and @adrianhopebailie. |
+1 to objects. With some minimal well defined structure and extensible. On Thu, Dec 17, 2015 at 12:17 PM, Dave Longley notifications@github.com
-Shane |
+1 to objects. The object can still contain a parameter (called name?) that is a string of value, "visa", "bitcoin", etc, but being an object will enable additional information to be associated with that payment method. My current thinking is having this an object would be much more preferable. |
Migrated to w3c/payment-request#48 |
The paymentRequest API asserts that the list of accepted payment methods that a payee exposes to the payer should be a list of strings:
http://wicg.github.io/paymentrequest/specs/paymentrequest.html#paymentrequest-interface
For example:
The Web Payments CG Browser API asserts that the list should be composed of objects:
http://wicg.github.io/web-payments-browser-api/#processing-a-payment-request
For example:
Should list of accepted payment methods be strings or objects?
The text was updated successfully, but these errors were encountered: