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 Additional Students Methods #60

Merged
merged 9 commits into from
Dec 19, 2023
Merged

Conversation

romandykyi
Copy link
Owner

This pull request adds additonal methods in UsersController and applies policy changes from pull request #56.

Added endpoints

  • GET api/users/students/[studentId]/groups
  • GET api/users/students/[studentId]/semesters

These methods allows administrators and teachers unrestricted access. For other users, access is granted only if their user ID matches the requested ID. They do not validate studentId(more information is available in swagger documentation).

Changes made

  • Added an authorization handler ViewStudentEnrollmentsAuthorizationHandler.

From pull request #56:

  • GET api/users/teachers method is now allowed for all registered users.
  • GET api/users/students is now allowed for teachers and administrators.

@romandykyi romandykyi merged commit 63db8ad into master Dec 19, 2023
1 check passed
@romandykyi romandykyi deleted the add-additional-students-methods branch December 19, 2023 13:58
@romandykyi romandykyi mentioned this pull request Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant