Implement the pipeline builder for section endpoints.

[mikehquan19]

Section aggregate endpoints (/section/courses & /section/professors) use 2 Mongo pipelines, sectionCoursePipeline & sectionProfessorPipeline for aggregating data.

They are functional, but are hardcoded in each endpoint. Since the 2 pipelines share lots of stage, it would be more managable for us to write a single function that returns the appropriate pipeline based on the types of endpoints.

Moreover, improve the pipeline to aggregate /section/courses & /section/:id/course so that it should return an array of courses for the former and only a single course for the latter (instead of an array of one course).

[Krish-Patel656]

I am new to the club, but I would like to try this one.

[mikehquan19]

Hi @Krish-Patel656 were you able to make changes to you pr so I can review?

[Krish-Patel656]

Hey, sorry for the late update. I've been pretty busy the past few weeks. I went ahead and made the changes, but noticed an Unable to create MongoDB client error. I noticed you mentioned that this would occur in the Discord, so I would apprelciate if you could take a quick look.

[mikehquan19]

Oh yes, the reason it happens is because the fork you submitted the PR from doesn't have the secrets required for the unit-tests, so the pipeline would fail. It shouldn't be a problem when i merged your code to the main repo tho. So as long as I review them and everything is alright, it's all good.

From now on, whatever new stuff you are working on, do it off of a branch of a main repo, so when you submit the PR, there won't be any Unable to create MongoDB client

[mikehquan19]

Just push the changes for merge conflict to your existing PR, and ignore the error, I will take care of the rest when I review them.

[Krish-Patel656]

Oh, ok, thanks for the clarification.

[mikehquan19]

@Krish-Patel656 Actually, can you do one thing for me? Tyler suggested this for smooth transition. Can you redirect to PR to branch 280-implement-pipeline-builder-for-section I just created?

So if your code is good I will just merge from 280-implement-pipeline-builder-for-section to develop. But if I need you to change something, you can work from the branch directly.

[Krish-Patel656]

I believe, I have correctly redirected the PR, please let me know if there are any issues or additional tasks I need to do

[mikehquan19]

Yup you're all good, for now just be on a lookout for any reviews i'll make

[Krish-Patel656]

Alright, thanks

[mikehquan19]

@Krish-Patel656 , I merged your code. Good works! One last thing I'd like you to work on is to merge sectionCourse() and sectionProfessor()? This is to reduce boilerplate code and make it more modular

The logic can be found on course and professor aggregate endpoints, it's not super complicated, just creating the map from the reflect.Type to the endpoints to pass to the pipeline builder.

After that, I'll close this issue once and for all