Services fix, tests #297
Merged
Services fix, tests #297
Conversation
This file contains 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
francisvaut
requested review from
EwoutV,
BramMeir,
bsilkyn,
tyboro2002,
DeLany123 and
Topvennie
* feat: academic year selector, better pagination and filtering composables * chore: added course pagination
* chore: celery #206 * chore: Support partial update for project * chore: support patch for project 2 * refactor!: rework checks and submissions * chore: allow models passed to tasks * feat: added -k to test.sh
EwoutV
approved these changes
Apr 12, 2024
DeLany123
pushed a commit
that referenced
this pull request
Apr 15, 2024
* test: create course * fix: send id to backend when creating faculty * test: create faculty * test: create group * test: cleanup create tests * test: assistant type * test: services and types folder * test: start with create project * test: cleanup setup file * test: cleanup request handlers * test: reset services + fix mistakes in teacher/assistant service * chore: fix linting * test: create project * chore: linting autofix * chore: bring back multipart/from-data * Better academic year selector, filtering and sorting (#292) * feat: academic year selector, better pagination and filtering composables * chore: added course pagination * Run submission checks (#268) * chore: celery #206 * chore: Support partial update for project * chore: support patch for project 2 * refactor!: rework checks and submissions * chore: allow models passed to tasks * feat: added -k to test.sh * fix: re-add translations * chore: display faculty in course edit * fix: courses to create projects filtered on selected calendar date * fix: make teacher command * fix: linting * test: admin delete * test: fix service mistake, assistant delete --------- Co-authored-by: Ewout Verlinde <[email protected]> Co-authored-by: Vincent Vallaeys <[email protected]> Co-authored-by: Bram Meir <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
getTeacherByCourse,getAssistantByCourseanddeleteAssistantservices