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.

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

Add "sections" aggregate endpoint to professor endpoint #232

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

Add "sections" aggregate endpoint to professor endpoint #232

jpahm opened this issue Oct 23, 2024 · 3 comments
Assignees
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 professor's sections.

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

  • Query based: GET /professor/sections (allows for querying like the base professor endpoint)
  • ID specific: GET /professor/{professorId}/sections (gets sections only for a specific professor)
@jpahm jpahm added the L2 A task suitable for someone who is comfortable implementing features. label Oct 23, 2024
@jpahm jpahm changed the title Add "sections" endpoint to professor endpoint Add "sections" aggregate endpoint to professor endpoint Oct 23, 2024
@jpahm jpahm transferred this issue from UTDNebula/api-tools Oct 24, 2024
@mikehquan19
Copy link
Contributor

I'd like to work on this so that all the endpoints tasks can be taken care of before new tasks pump out.

@jpahm
Copy link
Contributor Author

jpahm commented Nov 8, 2024

I'd like to work on this so that all the endpoints tasks can be taken care of before new tasks pump out.

That would be wonderful, thank you! New tasks are mainly going to be oriented around unit testing the tools (I'll be breaking up that big L3 task in the todo list) so they shouldn't interfere with the API too much.

In any case, this is all yours!

@mikehquan19
Copy link
Contributor

Hi @jpahm , I finally finished this task. Now I can move on and start working on the tasks of writing tests haha.

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L2 A task suitable for someone who is comfortable implementing features.
Projects
None yet
Development

No branches or pull requests

2 participants