Group querying for child / reply comments about software-engineering-daily-api

[sgundlapalli11]

I would like to take this on if possible.

Just to understand where the issue is, is this where we are looping through the parent comments?

[sgundlapalli11]

Still working on this, just haven't been able to dedicate much time to it. Just wanted to post here to keep it updated.

[jasonify]

Perfect and yeah exactly that :) . The last two functions on the Controller.model : https://github.com/SoftwareEngineeringDaily/software-engineering-daily-api/blob/5664f36eb976aabb76c58d03a4375b92fd5c680a/server/models/comment.model.js

Note the lean() call, since we modify those objects when populating the votes on each comment.

Most of this logic could also be moved from the controller to the model (like in the Post model).

[sgundlapalli11]

To approach this problem, I was going to add a method to take in all the parentComments and return childrenComments. I would make the query similar to this to return multiple documents?

Something I see we may have to do, is when we have all the childrenComments, we would then have to loop through them each one to add them to the appropriate parentComment which is happening here. I'm assuming that would still be preferred over sending a query for each parentComment though.

[jasonify]

👍 That's right, we just want to avoid the multiple queries