Course initial
Description
Closes #187 (closed), #183 (closed) A working backend support for courses. This includes a new repository and controller which manages the courses.
A course has the following structure (when serialized):
{
"id": "5cf028357fc2582206be877b",
"name": "Complexity Theory",
"courseCode": "TI3306",
"year": 2019,
"instructorSet": [],
"taSet": [
"ta"
],
"assignmentIds": [
"5ce653c5ff1dbf1ab679583d"
]
}
The following actions can be performed
- Retrieve the list of all courses
- Add a new course
- Add an assignment to a course
- Add a user to a course (with proper authentication)
Changes
- Refactored the getUser method to return an optional
- General refactoring of the DatabaseConnection to not throw more sensible exceptions (e.g. when a user cannot be found)
Additions
- A CourseRepository, Course and CourseController
Test and Review
To be filled in by the reviewers
-
All of the methods are commented to expectation -
The methods are tested to satisfaction -
There are no unnecessary files present in the MR -
The continuous integration has no problems with the MR -
The MR is filled in as requested (including labels, milestones, and reviewers) -
The documentation is up-to-date -
All nullable parameters are marked as such
Merge request reports
Activity
changed milestone to %User and course management
added typefeature-request + 1 deleted label
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
- Resolved by Tim van der Horst
added 138 commits
-
546de077...62694cbb - 137 commits from branch
development
- 201048b6 - Merge development into course-initial
-
546de077...62694cbb - 137 commits from branch
- Resolved by Tim van der Horst
I'm curious as to why ace is back in this MR. I don't see anything in the description of the MR.
added 22 commits
-
7a979f97...171323f7 - 21 commits from branch
development
- cf38de1a - Merge branches 'course-initial' and 'development' of...
-
7a979f97...171323f7 - 21 commits from branch
- Resolved by Tim van der Horst
added 1 commit
- ad4835fc - Fix secured methods not working by moving to secured service.
added 18 commits
-
ad4835fc...d95c110a - 17 commits from branch
development
- 07fa7ec5 - Merge development into course-initial
-
ad4835fc...d95c110a - 17 commits from branch
added 28 commits
-
07fa7ec5...4b10c872 - 26 commits from branch
development
- be5095b4 - Merge development into course-initial
- 996da2a5 - Revert dist.zip again
-
07fa7ec5...4b10c872 - 26 commits from branch
mentioned in commit de22a93f
unassigned @taivanderhorst