Re: [w3c/payment-request] Spread resolved `data` on updateWith() (#669)

domenic requested changes on this pull request.

So it's better to stay at the Infra level than to drop down to the ECMAScript level. Otherwise what happens here will be different if someone monkeys with Object.prototype, for example. You'll need to define some kind of ordered map merging operation, or let us do that for you in Infra.

BUT, as I pointed out in https://github.com/w3c/payment-request/issues/649#issuecomment-359108225, I don't think this patch is necessary at all. Let's continue discussing there.



-- 
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-request/pull/669#pullrequestreview-90265663

Received on Friday, 19 January 2018 22:33:54 UTC