Add MerchantValidationEvent.prototype.methodName

w3c / payment-request

Payment Request API

https://www.w3.org/TR/payment-request/

Other

489 stars 135 forks source link

Add MerchantValidationEvent.prototype.methodName #776

Closed marcoscaceres closed 5 years ago

marcoscaceres commented 5 years ago

closes #775

The following tasks have been completed:

[x] Confirmed there are no ReSpec errors/warnings.
[x] Modified Web platform tests
[x] Modified MDN Docs

Implementation commitment:

[x] Safari
[ ] Chrome
[x] Firefox
[ ] Edge (public signal)

Optional, Impact on Payment Handler spec?

Preview | Diff

marcoscaceres commented 5 years ago

Tests https://github.com/web-platform-tests/wpt/pull/12962

marcoscaceres commented 5 years ago

@aestes, @rsolomakhin, thanks also for the speedy review! Thanks @aestes for already adding the link to Safari!

@rsolomakhin, do you have a link too a Chrome bug?

romandev commented 5 years ago

@marcoscaceres I filed a bug instead of Rouslan. https://crbug.com/883605

marcoscaceres commented 5 years ago

Wrote some initial MDN docs for MerchantValidationEvent: https://developer.mozilla.org/en-US/docs/Web/API/MerchantValidationEvent

@aestes, do have development docs folks on the Safari side that can fill out the rest? Otherwise, we can get folks over on the Mozilla/MDN side to do it. However, want to make sure what is there is right, as it's (right now) very Apple Pay specific.

marcoscaceres commented 5 years ago

Ah, @romandev, you are the best. Thanks for filing it!