Re: [w3c/browser-payment-api] Make updateWith() reject the show() promise on validation failures (#451)

rsolomakhin commented on this pull request.



> +                            </li>
+                            <li>If the <a>additionalDisplayItems</a> member of
+                            <var>modifier</var> is present, then for each
+                            <a>PaymentItem</a> <var>item</var> in
+                            <var>modifier</var>.<a>additionalDisplayItems</a>:
+                              <ol>
+                                <li>Let <var>amountValue</var> be
+                                <var>item</var>.<a data-lt=
+                                "PaymentItem.amount">amount</a>.<a data-lt=
+                                "PaymentCurrencyAmount.value">value</a>.
+                                </li>
+                                <li>If <var>amountValue</var> is not a <a>valid
+                                decimal monetary value</a>, then <a>abort the
+                                update</a> with a <a>TypeError</a>.
+                                </li>
+                                <li>If the first character of

It should be OK to have negative line items in `additionalLineItems`. This can be "Debit card discount," for example. Let's reomve this `<li>...</li>`.

-- 
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/browser-payment-api/pull/451#pullrequestreview-26344106