Update code that is loading contacts by id directly from Pouch to use cht-datasource to load the contact data instead

[sugat009]

@shashi-sah2003
Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

[shashi-sah2003]

@jkuester @sugat009 is there any efficient way to find all the instances of loading contacts by id directly from Pouchdb.
one thing I can think of is that to search for const db in entire codebase, idk how good approach is this?

[shashi-sah2003]

@sugat009 @jkuester I think medic-client/contacts_* would be more suitable way to find Pouchdb direct query for contact? what do you guys thinks?

[sugat009]

@sugat009 @jkuester I think medic-client/contacts_* would be more suitable way to find Pouchdb direct query for contact? what do you guys thinks?

@shashi-sah2003 Yes, that would work but you will want to only search inside the shared-libs/lineage/src folder. I wrote this assuming that you are trying to search within the whole codebase. The codebase of lineage is just 2 maybe 3 files.

[shashi-sah2003]

@sugat009 the lineage folder has already been handled, the main file was hydration.js which was making direct DB call for contacts.

[sugat009]

@shashi-sah2003 I think I am misunderstanding something here.

the lineage folder has already been handled, the main file was hydration.js which was making direct DB call for contacts.

That is the goal of this task, right?

[shashi-sah2003]

@sugat009 okay so according to my understanding the lineage folder is one instance of direct DB call. But there can be other instances (different directory)also where PouchDB is directly being called. (Correct me if I'm wrong). So this issue is about finding all other instances except lineage directory where direct DB call is made as lineage directory has been taken care of in another PR.

[sugat009]

@shashi-sah2003 the scope of this project is to work within shared-libs/lineage. Other folders are out of scope for this ticket. To work on refactoring instances in the other folders, feel free to create another ticket.

[shashi-sah2003]

@sugat009 can't we modify the title of this issue? to something like outside of lineage library.