-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add payeeName input to spec #180
Conversation
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 % one nit; I think section 10.2 also needs updated to mention the name alongside the origin.
As usual, let's leave this for a week to let folks from the WPWG take a look and leave any feedback. I'll make sure its on the agenda for Thurs (cc @ianbjacobs ).
Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3
cc @tblachowicz |
Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3
Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3
Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3
Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3523272 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Ken Buchanan <kenrb@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/main@{#983464}
Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3523272 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Ken Buchanan <kenrb@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/main@{#983464}
Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3523272 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Ken Buchanan <kenrb@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/main@{#983464}
Thanks all, merging this one now! |
SHA: 654c78a Reason: push, by @nickburris Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Automatic update from web-platform-tests [SPC] Add payeeName option Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3523272 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Ken Buchanan <kenrb@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/main@{#983464} -- wpt-commits: 573fa63f75bbaa6940a8bfe9cb30bc06a212aae4 wpt-pr: 33213
Automatic update from web-platform-tests [SPC] Add payeeName option Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3523272 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Ken Buchanan <kenrb@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/main@{#983464} -- wpt-commits: 573fa63f75bbaa6940a8bfe9cb30bc06a212aae4 wpt-pr: 33213
Automatic update from web-platform-tests [SPC] Add payeeName option Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3523272 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Ken Buchanan <kenrb@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/main@{#983464} -- wpt-commits: 573fa63f75bbaa6940a8bfe9cb30bc06a212aae4 wpt-pr: 33213
Automatic update from web-platform-tests [SPC] Add payeeName option Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3523272 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Ken Buchanan <kenrb@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/main@{#983464} -- wpt-commits: 573fa63f75bbaa6940a8bfe9cb30bc06a212aae4 wpt-pr: 33213
Implementation for the recent spec change[1] to add a payeeName option as an optional alternative to payeeOrigin. The API now accepts one or both of payeeName and payeeOrigin. If both are provided, they are both shown in the UI with the format "payeeName (payeeOrigin)". This formatting when both fields are present is done in the UI code for two reasons; it's easier to test, and it can be considered a UI implementation detail how they are displayed, e.g. on a different platform or future iteration of the UI the origin could be displayed in a different font than the name, so the MVC model keeps them separate. [1] w3c/secure-payment-confirmation#180 Bug: 1298505 Change-Id: I19a255ce9f162b7d9e6c4cf0c2164242fa96cfc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3523272 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Ken Buchanan <kenrb@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/main@{#983464} NOKEYCHECK=True GitOrigin-RevId: f0b1a99a6af801ce883d2076103fb34e6fb1f65c
Per the proposal in #163, add a payeeName field to the spec.
Preview | Diff