Closed Bug 1175421 Opened 10 years ago Closed 10 years ago

[Contacts][NGA] Extract 'match' and 'showDuplicateContacts' from service_extensions.js

Categories

(Firefox OS Graveyard :: Gaia::Contacts, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(b2g-master fixed)

RESOLVED FIXED
FxOS-S1 (26Jun)
Tracking Status
b2g-master --- fixed

People

(Reporter: jorgep, Assigned: jorgep)

References

Details

(Whiteboard: [NG Gaia Contacts][patch])

Attachments

(1 file)

384 bytes, text/html
borjasalguero
: review+
arcturus
: review+
Details
To separate FB functionality from other functionalities, we are going to extract 'match' and 'showDuplicateContacts' from service_extensions.js to another class. These functions don't need FB to work.
Whiteboard: [NG Gaia Contacts]
Attached file 30622.html
Attachment #8623549 - Flags: review?(francisco)
Attachment #8623549 - Flags: review?(borja.bugzilla)
Attachment #8623549 - Flags: review?(borja.bugzilla) → review+
As NGA Program Manager suggested, let's replace the NGA-X milestones with FxOS-Sx ones (more generic ones), once Bug 1174794 has already landed
Target Milestone: NGA S3 (26Jun) → FxOS-S1 (26Jun)
Whiteboard: [NG Gaia Contacts] → [NG Gaia Contacts][patch]
Comment on attachment 8623549 [details] 30622.html Hi Jorge, looking good to me, just one comment on gh, related to when to load the js file. Right now is loaded always, and we could avoid to load it just for the list. The moment we go to detail or form then we could show it, wdyt?
Attachment #8623549 - Flags: review?(francisco) → review+
Hello Francisco, I've already done the changes :-) Ready to land! Thanks
Flags: needinfo?(francisco)
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: needinfo?(francisco)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: