Bug 989927 - [Contacts][Refactor] Create a library that reads the Global... #20423
Conversation
| dsList.forEach(function(aDs) { | ||
| datastores[aDs.owner] = aDs; | ||
| }); | ||
| datastores[MOZ_CONTACTS_OWNER] = new MozContactsDatastore(); |
There was a problem hiding this comment.
I guess this is transitional, we should mark it with the bug number that will solve this.
There was a problem hiding this comment.
oh absolutely
|
At the begining wasn't sure if adding the dependency of ther merger was right or not, sounds to me that there is a lot of code done (everything) that is reausable, and we have well separated the merging without any FB dependency so, does like a good option. One thing that would like to do, perhaps not in this PR, but as a follow up is reduce the number of lines of the method |
…bal Contacts DataStore and provide a single contact
|
Continuous Integration started. Results |
Bug 989927 - [Contacts][Refactor] Create a library that reads the Global...
... Contacts DataStore and provide a single contact