-
Notifications
You must be signed in to change notification settings - Fork 7
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
Secure Payment Confirmation to CR (2 changes since last review) 2023-01-11 > 2023-02-09 #201
Comments
Thanks @ianbjacobs for this request. Can you clarify your deadline? 1 February is a Wednesday and I18N meets on Thursdays. This only gives us a couple of weeks to perform the review and discuss any issues with you. If your date is semi-arbitrary, this may not matter. And, of course, we hope not to find any new issues :-). If you expect to submit a transreq on 1 Feburary, I'll try to expedite our review to the extent possible. |
Hi @aphillips, Thanks for the quick reply. We have flexibility on the date. Would 9 February work for you? (Assuming you meet that day.) |
@ianbjacobs Thanks! That's super helpful. We'll work to finish up by the 2nd (before if possible). |
@ianbjacobs I just filed the two issues we found, one of which is editorial. The other is thorny only from the point of view that you need to progress your spec and I still don't have a good answer for language and direction metadata for |
Thanks, @aphillips. See:
|
Hi @aphillips, |
Hi @aphillips and @xfq, Thank you again for the review. We have completed merging all pull requests resulting from the review and thus I believe this issue can be closed. All the best, Ian |
Name of your specification
Secure Payment Confirmation
URL of your specification
https://w3c.github.io/secure-payment-confirmation/
When does the review need to be finished?
2023-02-01
What has changed since any previous review?
In August 2022 the Web Payments Working Group requested pre-Candidate Recommendation horizontal review of Secure Payment Confirmation (SPC). All reviews led to satisfactory outcomes. Thank you for the I18N review:
#190
Since then, the Web Payments Working Group has made or plans to make two non-editorial changes to the specification for which we seek additional horizontal review:
specification. Experimentation with this feature has demonstrated its utility to at least one organization that has experimented with SPC.
Please point to the results of your own self-review
w3c/secure-payment-confirmation#202
Where and how should the i18n WG raise issues?
https://github.com/w3c/secure-payment-confirmation/issues/
Pointer to any explainer for the spec
https://github.com/w3c/secure-payment-confirmation/blob/main/explainer.md
Other comments
Here are some observations about how these changes relate to previously raised issues; see
https://github.com/w3c/secure-payment-confirmation/issues?q=is%3Aclosed+label%3Ai18n-needs-resolution
Opt-out in the UX. Previous issue on locale hint is relevant:
[Spec] Add SecurePaymentConfirmationRequest.locale secure-payment-confirmation#212
Presumably the browser would use the same locale hint for the text informing
the user about the ability to opt out.
There are no new input fields to the API that result in displayable strings. Thus,
this issue is not relevant:
DOMString
forpayeeName
vs.USVString
for other fields? secure-payment-confirmation#208There is no new example text. Thus this issue is not relevant:
Error example contains a hardcoded string secure-payment-confirmation#206
Note that the previous issue on lang/dir metadata has been left open for ongoing TC39 discussions, without objections to the WPWG advancing SPC to CR:
w3c/secure-payment-confirmation#205 (comment)
Thank you!
The text was updated successfully, but these errors were encountered: