Fixes for MultipleChoiceOption and FeedbackOption API
All threads resolved!
All threads resolved!
This merge request fixes two things:
- Deleting a
FeedbackOption
will now throw an error if it is coupled to aMultipleChoiceOption
- Inserting a
MultipleChoiceOption
will now check theproblem_id
parameter to see if an actualProblem
with the specifiedproblem_id
exists.
Fixes #51 (closed) and #55 (closed)
Note: It might be useful to merge !16 (merged) first, since I can write test code for this MR.
Edited by Ghost User
Merge request reports
Activity
- Resolved by Hidde Leistra
Fixes #51 (closed) and #55 (closed)
added 21 commits
-
551d9807...ea611c95 - 20 commits from branch
develop
- b63c9fa8 - Fixed conflicts, and added test for feedback deletion
-
551d9807...ea611c95 - 20 commits from branch
added 1 commit
- 5eae9d3b - Changed added test for deleting feedback options, improved mc_options test
added 33 commits
-
60181548...9df3d26b - 32 commits from branch
develop
- 4e06a1f0 - Resolved conflicts with develop
-
60181548...9df3d26b - 32 commits from branch
An error notification still needs to be shown in the frontend.
Continues in #52 (closed).
mentioned in commit 78196d7b
Please register or sign in to reply