Clear the field-specific merchant error invalid state when the field has been edited

VERIFIED FIXED in Firefox 64

Status

()

P1
normal
VERIFIED FIXED
7 months ago
6 months ago

People

(Reporter: MattN, Assigned: jaws)

Tracking

Trunk
Firefox 64
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox64 verified)

Details

(Whiteboard: [webpayments])

Attachments

(2 attachments)

"Once the user begins to type into a field with an in-context error, the error will disappear. This will also work if a user deletes text within the field."

https://mozilla.invisionapp.com/share/YAFI31XR3KP#/screens/275361828

Right now https://dxr.mozilla.org/mozilla-central/rev/4e56a2f51ad739ca52046723448f3129a58f1666/browser/components/payments/test/browser/browser_shippingaddresschange_error.js#158 will hang after bug 1476345 disables the save button if there are invalid field because setCustomValidity is called with merchant-specific errors call but they are never cleared.
Flags: qe-verify+
Priority: -- → P2
QA Contact: hani.yacoub
Whiteboard: [webpayments] [triage] → [webpayments-reserve]
Whiteboard: [webpayments-reserve] → [webpayments]
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Priority: P2 → P1
Comment on attachment 9006701 [details]
Bug 1482808 - Clear the field-specific merchant error invalid state when the field has been edited. r?sfoster

Sam Foster [:sfoster] has approved the revision.
Attachment #9006701 - Flags: review+

Comment 3

7 months ago
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f669be991db6
Clear the field-specific merchant error invalid state when the field has been edited. r=sfoster

Comment 4

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f669be991db6
Status: ASSIGNED → RESOLVED
Last Resolved: 7 months ago
status-firefox64: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64

Comment 5

7 months ago
Posted image error.png
I'm not sure when the in-context error will be displayed, to try to verify when the error will disappear.

I accessed "https://rsolomakhin.github.io/pr/single/" (FTU), entered valid data in every field except "ZIP Code" field, no in-context error is displayed, only a red outline. (Please see screenshot attached)
Flags: needinfo?(jaws)
This bug only focused on shipping address errors that are supplied by the merchant after the user changes their shipping address. I'm not sure of a website that implements this yet because it was just added to the spec.

The error text for the zip code will appear in a tooltip if you hover over the field. We should probably have that message show up in a popup the same way as the merchant specific address fields. I've filed bug 1490077 to display these the same.
Flags: needinfo?(jaws)

Comment 7

6 months ago
Since bug 1490077 introduced the field-specific errors, the red popup will disappear after the user edited the incorrect data.

The same behavior was observed for the merchant errors that appear on the summary page after editing the invalid Shipping address or saved CC. 

Is this check enough to cover this issue fix?
Flags: needinfo?(jaws)

Comment 9

6 months ago
Thanks Jared!

Based on Comment 7 and Comment 8, I will mark this issues as Verified - Fixed  on Windows 10/7, Mac OS 10.13 and Ubuntu 16.04.
Status: RESOLVED → VERIFIED
status-firefox64: fixed → verified
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.