Closed Bug 1023764 Opened 10 years ago Closed 10 years ago

[Loop] Keep Contacts Info updated

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: borjasalguero, Assigned: ferjm)

References

Details

Attachments

(1 file)

60 bytes, text/x-github-pull-request
jaoo
: review+
borjasalguero
: review+
borjasalguero
: feedback+
Details | Review
We need to keep a cached version of the contacts we are using in our call log, but we need to update this info in our IndexedDB if any change happens in the Contact List.

As well we will create a ContactsHelper for retrieving the info of a contact given an ID, and other methods related.
Depends on: 1023760
Blocks: 1007937
Assignee: nobody → ferjmoreno
Attached file WIP
Attachment #8455465 - Flags: review?(josea.olivera)
Attachment #8455465 - Flags: review?(borja.bugzilla)
Comment on attachment 8455465 [details] [review]
WIP

LGTM. r=me

I'm not very familiar with the call log/contact stuff. I would prefer Borja to take a look also before landing this.

Thanks Fernando!
Attachment #8455465 - Flags: review?(josea.olivera) → review+
(In reply to José Antonio Olivera Ortega [:jaoo] from comment #3)
> Comment on attachment 8455465 [details] [review]
> WIP
> 
> LGTM. r=me
> 
> I'm not very familiar with the call log/contact stuff. I would prefer Borja
> to take a look also before landing this.

Oops, forgot to comment that do not forget to rebase this work as we already landed the bumping up version logic. Once you rebase, the version should be bumped up.
Comment on attachment 8455465 [details] [review]
WIP

I think that there are some things to improve, so I've added some comments in Github. I'm keeping the r? flag in order to recheck this when ready. Thanks Fernando!
Attachment #8455465 - Flags: feedback+
Attachment #8455465 - Flags: review?(borja.bugzilla)
Attachment #8455465 - Flags: review?(borja.bugzilla)
Comment on attachment 8455465 [details] [review]
WIP

This patch is working perfectly! R+!
Attachment #8455465 - Flags: review?(borja.bugzilla) → review+
https://github.com/mozilla-b2g/firefoxos-loop-client/commit/07f04cacb73b5f5cd83e242780c895cd0f623f95
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: