search

w3c / payment-request

Payment Request API
https://www.w3.org/TR/payment-request/
Other
489 stars 135 forks source link

Editorial: fixup updateWith() example #667

Closed marcoscaceres closed 6 years ago

marcoscaceres commented 6 years ago

Remove mention of non-existent [[updatedUI]] internal slot. Simplify language a little bit.

Preview | Diff

marcoscaceres commented 6 years ago

@ianbjacobs, could you please send a request to republish the spec?

ianbjacobs commented 6 years ago

@marcoscaceres,

Sorry for the delay. I see that a number of (mostly editorial) changes have happened to the main branch since the previous CR update (on 21 Sep 2017 [1]). Should I include all of them? or just this one (#667), or all of them up to and including this one?

Ian

[1] https://www.w3.org/TR/payment-request/

marcoscaceres commented 6 years ago

All changes would be great 👍

ianbjacobs commented 6 years ago

@marcoscaceres,

Here's the draft I was looking at: https://rawgit.com/w3c/payment-request/876a9cc21836c7d2b0fe27f17837cee7da63b664/index.html?specStatus=CR

There are 2 respec warnings: Found linkless element with text 'displayitems' but no matching . Found linkless element with text 'additionaldisplayitems' but no matching .

Would you mind fixing those first? Thanks!

Ian

marcoscaceres commented 6 years ago

Would you mind fixing those first? Thanks!

Fixed! Thanks for letting me know.

PS: Alternative is to grab the tip from https://w3c.github.io/payment-request/?specStatus=CR

ianbjacobs commented 6 years ago

Thanks; will send publication request. Ian

marcoscaceres commented 6 years ago

@ianbjacobs let's get https://github.com/w3c/payment-request/pull/671 merged first :)