Add "courses" aggregate endpoint to professor endpoint

UTDNebula / nebula-api

The central API for Nebula Labs. Makes UTD data easily available through endpoints for both internal and public usage.

MIT License

26 stars 33 forks source link

Add "courses" aggregate endpoint to professor endpoint #230

Closed jpahm closed 6 days ago

jpahm commented 1 month ago

Currently, it can be inconvenient and overcomplicated for our end users to perform some common aggregate operations such as getting all of a professor's courses.

As such, we should add the following endpoints which immediately aggregate and return matching professor's courses:

Query based: GET /professor/courses (allows for querying like the base professor endpoint)
ID specific: GET /professor/{professorId}/courses (gets courses only for a specific professor)

mikehquan19 commented 1 month ago

Can I work on this? I want to practice querying the MongoDB database and building API a bit more.

jpahm commented 1 month ago

Can I work on this? I want to practice querying the MongoDB database and building API a bit more.

You're more than welcome to!

mikehquan19 commented 4 weeks ago

Hi! I've made the pull request. Hope you will review it and give me feedback.

Thank you.

jpahm commented 3 weeks ago

Thanks, I will review it shortly!

mikehquan19 commented 1 week ago

Hi @jpahm, I've fixed the problem about options limit, so I just want to let you know in case you haven't been aware of it yet. Thank you.

jpahm commented 1 week ago

Hi @jpahm, I've fixed the problem about options limit, so I just want to let you know in case you haven't been aware of it yet. Thank you.

Thank you, I will take a look!