Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add "professors" aggregate endpoint to course endpoint #228

Open
jpahm opened this issue Oct 23, 2024 · 5 comments
Open

Add "professors" aggregate endpoint to course endpoint #228

jpahm opened this issue Oct 23, 2024 · 5 comments
Assignees
@mohammadmehrab
Labels
L2 A task suitable for someone who is comfortable implementing features.

Comments

@jpahm
Copy link
Contributor

jpahm commented Oct 23, 2024

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

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

  • Query based: GET /course/professors (allows for querying like the base course endpoint)
  • ID specific: GET /course/{courseId}/professors (gets professors only for a specific course)
@jpahm jpahm added the L2 A task suitable for someone who is comfortable implementing features. label Oct 23, 2024
@mohammadmehrab
Copy link
Contributor

Can I be assigned this

@jpahm
Copy link
Contributor Author

jpahm commented Oct 31, 2024

Can I be assigned this

Absolutely!

@jpahm
Copy link
Contributor Author

jpahm commented Nov 21, 2024

Can I be assigned this

Hey, how's this going?

@mohammadmehrab
Copy link
Contributor

Hey, I'm hoping to finish this by Saturday. I've been swamped with exams, HackUTD, and a bunch of other stuff, so I haven't gotten to finishing this

@jpahm
Copy link
Contributor Author

jpahm commented Nov 21, 2024

Hey, I'm hoping to finish this by Saturday. I've been swamped with exams, HackUTD, and a bunch of other stuff, so I haven't gotten to finishing this

No worries, thanks for the update :)

@mohammadmehrab mohammadmehrab mentioned this issue Nov 26, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mohammadmehrab mohammadmehrab

Labels
L2 A task suitable for someone who is comfortable implementing features.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jpahm @mohammadmehrab