-
Notifications
You must be signed in to change notification settings - Fork 6
Add backend routes for courses and classes #1013
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
Merged
lhvy
merged 22 commits into
server-rewrite
from
1000-update-backend-route-for-courses-and-classes
Jul 17, 2025
Merged
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
f6849f0
Add routes for adding and removing course
jason4193 b5f21bf
Update routes by moving validation to controller
jason4193 90a36c3
Add route for update course colour
jason4193 7b086e5
add OpenAPI for api-docs for backend using swagger library
jason4193 f10340d
Add ApiProperty for user's route
jason4193 8351ba5
Add get route for courses and classes with update of routing urls
jason4193 7a5767b
Update PUT request for setCourseColour to PATCH request
jason4193 5c78771
Update user routes to using AuthenticatedRequest Type
jason4193 b0adab4
Add selectClass & removeClass route
jason4193 48c8a21
Remove OpenAPI and Swagger Library, switched to use Postman
jason4193 6ba206d
Update route for select and remove class
jason4193 738fde3
Remove unwanted routes with forwarding functions
jason4193 7ddde09
Resolve usage of null and undefined
jason4193 9249ba9
Resolve PR review comments
jason4193 cb14284
Slim down the Course and Class detail for graphql, and add validation…
jason4193 ac96945
Update routes returning status code
jason4193 c7e9e79
Refactor user.controller, move all business validation into service l…
jason4193 9a98db7
Remove duplicate getCourse request in addSelectedClass
jason4193 984c7e1
Resolve issues on the PR code review
jason4193 cbc8ec2
Resolve minor issue on function call, optimise prisma call
jason4193 20fea88
Resolve function name typo
jason4193 10326d7
Update service level function name corresponds to control level
jason4193 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| export class ClassDetails { | ||
| activity: string; | ||
| section: string; | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,10 +1,11 @@ | ||
| import { Module } from '@nestjs/common'; | ||
| import { GraphqlService } from 'src/graphql/graphql.service'; | ||
| import { PrismaService } from 'src/prisma/prisma.service'; | ||
| import { UserService } from './user.service'; | ||
| import { UserController } from './user.controller'; | ||
|
|
||
| @Module({ | ||
| providers: [UserService, PrismaService], | ||
| providers: [UserService, PrismaService, GraphqlService], | ||
| controllers: [UserController], | ||
| }) | ||
| export class UserModule {} |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.