Re: [w3c/payment-handler] Add userHint to PaymentManager (#206)

romandev commented on this pull request.



> @@ -529,6 +529,7 @@
       interface PaymentManager {
         [SameObject] readonly attribute PaymentInstruments instruments;
         [Exposed=Window] static Promise<PermissionState> requestPermission();
+        DOMString userHint;     

I think this should be attribute because this is not dictionary type.
```attribute DOMString userHint```

Also, if you don't mind, isn't it better to move this attribute to PaymentInstruments?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/w3c/payment-handler/pull/206#pullrequestreview-59622037

Received on Wednesday, 30 August 2017 16:49:49 UTC