Closed Bug 1429207 Opened 3 years ago Closed 3 years ago

Implement the "failure" screen for when the merchant rejects the payment and doesn't request retrying

Categories

(Firefox :: WebPayments UI, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 60
Tracking Status
firefox60 --- fixed

People

(Reporter: MattN, Assigned: jaws)

References

()

Details

(Whiteboard: [webpayments])

Attachments

(1 file)

After sending the PaymentResponse to the merchant via the DOM code, the merchant responds to indicate if the payment was accepted or rejected. If the front-end code receives `nsIPaymentActionResponse.PAYMENT_REJECTED` (retrying isn't supported) then failure UI should appear.

Retry (error recovery) support is being discussed in https://github.com/w3c/payment-request/issues/647
See Also: → 1429213
Priority: P3 → P1
Depends on: 1435163
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Comment on attachment 8954636 [details]
Bug 1429207 - Implement the 'failure' screen for when the merchant rejects the payment.

https://reviewboard.mozilla.org/r/223724/#review229768

Thanks

::: toolkit/components/payments/test/mochitest/test_payment_dialog.html:126
(Diff revision 1)
> -  await el1.requestStore.setState({completionState: "processing"});
> -  await asyncElementRendered();
> +  for (let [completionState, label] of completionStates) {
> +    await el1.requestStore.setState({completionState});
> -  is(payButton.textContent, "Processing", "Check processing label");
> -  ok(!payButton.disabled, "Button is still enabled");
> -  await el1.requestStore.setState({completionState: "success"});
> -  await asyncElementRendered();
> +    await asyncElementRendered();
> -  is(payButton.textContent, "Done", "Check success label");
> +    is(payButton.textContent, label, "Check payButton label");
> -  ok(!payButton.disabled, "Button is still enabled");
> +    ok(!payButton.disabled, "Button is still enabled");
> +  }

This also works (re: my review of success).
Attachment #8954636 - Flags: review?(MattN+bmo) → review-
Comment on attachment 8954636 [details]
Bug 1429207 - Implement the 'failure' screen for when the merchant rejects the payment.

https://reviewboard.mozilla.org/r/223724/#review229770

Oops
Attachment #8954636 - Flags: review- → review+
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e1b30d54da0b
Implement the 'failure' screen for when the merchant rejects the payment. r=MattN
https://hg.mozilla.org/mozilla-central/rev/e1b30d54da0b
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Whiteboard: [webpayments]
Product: Toolkit → Firefox
Target Milestone: mozilla60 → Firefox 60
You need to log in before you can comment on or make changes to this bug.