Skip to content
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

Validate payment method data #381

Closed
wants to merge 2 commits into from

Conversation

domenic
Copy link
Collaborator

@domenic domenic commented Dec 14, 2016

This is part of w3c/payment-method-basic-card#20. What remains is for the basic-card specification to define its validate payment method data algorithm.


This is on top of #374 and should not be merged until that is.

- Fixes w3c#373: align ordering of steps with Blink's implementation, except check for allowed-to-use first since that makes more sense
- Fixes w3c#334: properly get the total.amount.value instead of pretending total is a string
- Fixes w3c#333 and fixes w3c#335: do not assume various dictionaries are present
- Fixes part of w3c#321: use proper dictionary terminology
- Ensures request.[[details]].shippingOptions/modifiers is always a sequence, instead of sometimes left as not present
- Fixes various algorithm structure and typographic issues
This is part of w3c/payment-method-basic-card#20. What remains is for the basic-card specification to define its validate payment method data algorithm.
@domenic
Copy link
Collaborator Author

domenic commented Dec 14, 2016

Closing since per #307 (comment) this cannot be done synchronously; it needs to happen in another process.

@domenic domenic closed this Dec 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant