Merge pull request learningequality#11937 from nucleogenesis/11017--r…

…eplace-questions Quiz creation: Question replacement & a bevy of misc fixes and polish
jredrejo · Apr 18, 2024 · afe5a61 · afe5a61
2 parents 7f7032a + 4cf2258
commit afe5a61
Show file tree

Hide file tree

Showing 18 changed files with 1,020 additions and 451 deletions.
diff --git a/kolibri/plugins/coach/assets/src/composables/quizCreationSpecs.js b/kolibri/plugins/coach/assets/src/composables/quizCreationSpecs.js
@@ -9,28 +9,28 @@
 /**
  * @typedef   {Object}  QuizExercise    An object referencing an exercise or topic to be used
  *                                      within the `QuizSeciton.resource_pool` property.
+ * @property  {string}  id              Unique ID for this exercise (aka, `exercise_id` elsewhere)
  * @property  {string}  title           The resource title
  * @property  {string}  ancestor_id     The ID of the parent contentnode
  * @property  {string}  content_id      The ID for the piece of content
- * @property  {string}  id              Unique ID for this exercise
  * @property  {bool}    is_leaf         Whether or not this is a leaf node (i.e. an exercise)
  * @property  {string}  kind            Exercise or Topic in our case - see: `ContentNodeKinds`
  */
 
 export const QuizExercise = {
-  title: {
+  id: {
     type: String,
     default: '',
   },
-  ancestor_id: {
+  title: {
     type: String,
     default: '',
   },
-  content_id: {
+  ancestor_id: {
     type: String,
     default: '',
   },
-  id: {
+  content_id: {
     type: String,
     default: '',
   },
@@ -55,6 +55,8 @@ export const QuizExercise = {
 /**
  * @typedef  {Object} QuizQuestion         A particular question in a Quiz - aka an assessment item
  *                                         from an QuizExercise.
+ * @property {string} id                   A  ** unique **  identifier for this question that is
+ *                                          a combination of <exercise_id>:<question_id>
  * @property {string} exercise_id          The ID of the resource from which the question originates
  * @property {string} question_id          A *unique* identifier of this particular question within
  *                                         the quiz -- same as the `assessment_item_id`
@@ -63,6 +65,10 @@ export const QuizExercise = {
  *                                         same exercise title to differentiate them
  */
 export const QuizQuestion = {
+  id: {
+    type: String,
+    required: true,
+  },
   exercise_id: {
     type: String,
     required: true,