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)
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)
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.
| Assignee | ||
Updated•10 years ago
|
Whiteboard: [NG Gaia Contacts]
| Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8623549 -
Flags: review?(francisco)
Attachment #8623549 -
Flags: review?(borja.bugzilla)
Updated•10 years ago
|
Attachment #8623549 -
Flags: review?(borja.bugzilla) → review+
Comment 2•10 years ago
|
||
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)
Updated•10 years ago
|
Whiteboard: [NG Gaia Contacts] → [NG Gaia Contacts][patch]
Comment 3•10 years ago
|
||
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+
| Assignee | ||
Comment 4•10 years ago
|
||
Hello Francisco,
I've already done the changes :-)
Ready to land!
Thanks
Flags: needinfo?(francisco)
Comment 5•10 years ago
|
||
Thanks for the work Jorge,
landed:
https://github.com/mozilla-b2g/gaia/commit/88a85959a2034d1f073e8e5601254c159f35448f
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: needinfo?(francisco)
Resolution: --- → FIXED
Updated•10 years ago
|
status-b2g-master:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•