w3c / i18n-request

Horizontal review requests are made via issues in this repo, and tracked by our radar.
https://github.com/orgs/w3c/projects/91/views/1
1 stars 7 forks source link

Secure Payment Confirmation to CR (2 changes since last review) 2023-01-11 > 2023-02-09 #201

Closed ianbjacobs closed 1 year ago

ianbjacobs commented 1 year ago

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: https://github.com/w3c/i18n-request/issues/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:

Please point to the results of your own self-review

https://github.com/w3c/secure-payment-confirmation/issues/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

1) Opt-out in the UX. Previous issue on locale hint is relevant: https://github.com/w3c/secure-payment-confirmation/pull/212

Presumably the browser would use the same locale hint for the text informing
the user about the ability to opt out.

2) There are no new input fields to the API that result in displayable strings. Thus, this issue is not relevant: https://github.com/w3c/secure-payment-confirmation/issues/208

3) There is no new example text. Thus this issue is not relevant: https://github.com/w3c/secure-payment-confirmation/issues/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: https://github.com/w3c/secure-payment-confirmation/issues/205#issuecomment-1255167247

Thank you!

aphillips commented 1 year ago

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.

ianbjacobs commented 1 year ago

Hi @aphillips,

Thanks for the quick reply. We have flexibility on the date. Would 9 February work for you? (Assuming you meet that day.)

aphillips commented 1 year ago

@ianbjacobs Thanks! That's super helpful. We'll work to finish up by the 2nd (before if possible).

aphillips commented 1 year ago

@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 displayName. If necessary, let's chat about what (if anything) your spec needs to say about this.

ianbjacobs commented 1 year ago

Thanks, @aphillips. See:

1) Pull request to add locale to an example (editorial) https://github.com/w3c/secure-payment-confirmation/pull/229

2) Questions about a suggested Note: https://github.com/w3c/secure-payment-confirmation/issues/205#issuecomment-1408777086

ianbjacobs commented 1 year ago

Hi @aphillips, I believe we have address the issues raised by the group to everyone's satisfaction. Please confirm that the I18N folks are satisfied with the changes, and thanks to all for helping to improve the spec!

ianbjacobs commented 1 year ago

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