-
Notifications
You must be signed in to change notification settings - Fork 42
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
Remove support for address, contact info from PR API (align with pull request 955) #389
Conversation
2) Removed shipping, contact information to align with PR 955 on PR API w3c/payment-request#955
updated tidyconfig to align with PR API version
A note on this pull request: There are some change indications that are not areas that I edited. I'm not sure if this is due to some text not having been tidied before, or new tidy behavior. Sorry for that noise. |
More notes:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@rsolomakhin, does anything need to happen on the Chrome side by virtue of updating the spec this way (e.g., bug tracker as was done for the PR API edit)? |
Filed https://crbug.com/1207601. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Drop PaymentAddress, shipping + billing address support payment-request#955
The following tasks have been completed:
Implementation commitment:
Preview | Diff