- From: Dave Longley <notifications@github.com>
- Date: Mon, 28 Aug 2017 03:39:29 +0000 (UTC)
- To: w3c/payment-request <payment-request@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Monday, 28 August 2017 03:39:52 UTC
dlongley commented on this pull request.
> + need to call <a>updateWith()</a> and provide a
+ <a>PaymentDetailsUpdate</a> dictionary, or a promise for one,
+ containing changed values that the <a>user agent</a> presents to
+ the user.
+ </p>
+ <p>
+ To prevent the user interface from blocking (and to reflect
+ changes made by the end-user through the UI), developers need to
+ immediately call <a>updateWith()</a>. If they don't,
+ <a>[[\didUpdate]]</a> becomes true and <a>updateWith()</a>
+ will throw.
+ </p>
+ <pre class="example" title="how to use `updateWith()` correctly.">
+ // ❌ Bad - this won't work!
+ request.onshippingaddresschange = async ev => {
+ // `await` goes to next tick, so [[didUpdate]] is now true.
Could flip it and go with "canUpdate" or "updatePermitted" and turn it off after the opportunity to update is lost or has already been taken.
--
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/591#discussion_r135436885
Received on Monday, 28 August 2017 03:39:52 UTC