Payer/contact picker dropdown labels need to take @address-fields into account

VERIFIED FIXED in Firefox 63

Status

()

enhancement
P1
normal
VERIFIED FIXED
9 months ago
8 months ago

People

(Reporter: MattN, Assigned: prathiksha)

Tracking

Trunk
Firefox 63
Points:
---

Firefox Tracking Flags

(firefox63 verified)

Details

(Whiteboard: [webpayments])

Attachments

(1 attachment)

The dropdown labels on the address-picker currently use the usual address labels including two columns but they should take @address-fields into account to only show relevant data in the dropdowns.
(Assignee)

Updated

9 months ago
Assignee: nobody → prathikshaprasadsuman
Status: NEW → ASSIGNED
Priority: P2 → P1
Whiteboard: [webpayments] → [webpayments-reserve]
Flags: qe-verify?
Whiteboard: [webpayments-reserve] → [webpayments]
Flags: qe-verify? → qe-verify+
QA Contact: hani.yacoub
This is blocked on answers to the questions in bug 1463545 comment 4. Someone could start to implement the filtering based on @address-fields but how we handle missing values for those fields (in both the open and closed states) is what needs an answer.
Comment on attachment 8998124 [details]
Bug 1475760 - Take address-fields into account for payer/contact picker dropdown labels. r?MattN

Matthew N. [:MattN] (PM if requests are blocking you) has approved the revision.
Attachment #8998124 - Flags: review+

Comment 4

8 months ago
Pushed by mozilla@noorenberghe.ca:
https://hg.mozilla.org/integration/mozilla-inbound/rev/db83318361c5
Take address-fields into account for payer/contact picker dropdown labels. r=MattN

Comment 5

8 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/db83318361c5
Status: ASSIGNED → RESOLVED
Last Resolved: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63

Comment 6

8 months ago
Build ID: 20180821100053
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:63.0) Gecko/20100101 Firefox/63.0

Verified as fixed on Firefox Nightly 63.0a1 on Windows 10 x 64, Windows 7 x32, Mac OS X 10.13 and Ubuntu 17.04 x64.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.