Closed
Bug 1023764
Opened 11 years ago
Closed 11 years ago
[Loop] Keep Contacts Info updated
Categories
(Firefox OS Graveyard :: Gaia::Loop, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: borjasalguero, Assigned: ferjm)
References
Details
Attachments
(1 file)
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.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → ferjmoreno
Assignee | ||
Comment 2•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Attachment #8455465 -
Flags: review?(josea.olivera)
Attachment #8455465 -
Flags: review?(borja.bugzilla)
Comment 3•11 years ago
|
||
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+
Comment 4•11 years ago
|
||
(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.
Reporter | ||
Comment 5•11 years ago
|
||
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+
Assignee | ||
Updated•11 years ago
|
Attachment #8455465 -
Flags: review?(borja.bugzilla)
Assignee | ||
Updated•11 years ago
|
Attachment #8455465 -
Flags: review?(borja.bugzilla)
Reporter | ||
Comment 6•11 years ago
|
||
Comment on attachment 8455465 [details] [review]
WIP
This patch is working perfectly! R+!
Attachment #8455465 -
Flags: review?(borja.bugzilla) → review+
Assignee | ||
Comment 7•11 years ago
|
||
https://github.com/mozilla-b2g/firefoxos-loop-client/commit/07f04cacb73b5f5cd83e242780c895cd0f623f95
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•