Closed Bug 889756 Opened 11 years ago Closed 11 years ago

Design and implement a ContactsMatcher module (passive mode)

Categories

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

x86
macOS
defect

Tracking

(blocking-b2g:koi+)

RESOLVED FIXED
1.1 QE4 (15jul)
blocking-b2g koi+

People

(Reporter: jmcf, Assigned: jmcf)

References

Details

(Whiteboard: [u=commsapps-user c=contacts p=4])

Attachments

(1 file)

Given a new Contact C the ContactsMatcher module will be responsible for finding another Contact C2 that will be a candidate to be merged or linked with C.
Blocks: 875793
blocking-b2g: --- → koi?
Whiteboard: [u=commsapps-user c=contacts p=0]
Whiteboard: [u=commsapps-user c=contacts p=0] → [u=commsapps-user c=contacts p=4]
Whiteboard: [u=commsapps-user c=contacts p=4] → [u=commsapps-user c=contact p=4]
Whiteboard: [u=commsapps-user c=contact p=4] → [u=commsapps-user c=contacts p=4]
listed as must have in v1.2 for COMM team. koi+
blocking-b2g: koi? → koi+
Attached file Pointer to GH PR
First version. implements silent mode matcher with tests
Attachment #775582 - Flags: review?(francisco.jordano)
Attachment #775582 - Flags: review?(crdlc)
Status: NEW → ASSIGNED
Comment on attachment 775582 [details]
Pointer to GH PR

Great work!! In general, for me, it is ok! Although there are some comments on Gitbub related to refactor most of them. Take a look when you want because I've just finished. Thanks Jose
Attachment #775582 - Flags: review?(crdlc) → review+
Comment on attachment 775582 [details]
Pointer to GH PR

Amazing job!

r+ with Cristian's comments and some of the comments I let you in github. Also could we add a bit of documentation for the methods in general?

Thanks again!
Attachment #775582 - Flags: review?(francisco.jordano) → review+
https://github.com/mozilla-b2g/gaia/commit/f2e2403873bcd83b046ff2b9baf61e8db6224496
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Summary: Design and implement a ContactsMatcher module → Design and implement a ContactsMatcher module for the passive mode
Summary: Design and implement a ContactsMatcher module for the passive mode → Design and implement a ContactsMatcher module (passive mode)
Blocks: 895799
No longer blocks: 875793
Depends on: 897864
Depends on: 898345
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: