Merge pull request #110 from CS3219-AY2425S1/aiden-question-fixes

Change error messages

peer-prep-fe/src/add-page/add-page.component.html

@@ -93,6 +93,6 @@ <h2>CATEGORY</h2>
   </div>
 
   <div class="saving-button">
-    <button class="save-button" (click)="saveQuestion()">SAVE</button>
+    <button class="save-button" (click)="saveQuestion()" [disabled]="questionForm.invalid">SAVE</button>
   </div>
 </div>

peer-prep-fe/src/add-page/add-page.component.ts

@@ -79,6 +79,13 @@ export class AddPageComponent {
       this.questionForm.markAllAsTouched()
       return
     }
+
+    const selectedCategories = this.question_categories.filter(cat => cat.selected);
+    if (selectedCategories.length === 0) {
+      alert("Please select at least one category.");
+      return;
+    }
+
     const newQuestion = {
       Question_title: this.questionForm.value.question_title,
       Question_description: this.questionForm.value.question_description,
@@ -94,7 +101,7 @@ export class AddPageComponent {
         this.onAddComplete()
       },
       (error) => {
-        alert("Error adding question")
+        alert("Question with the same title already exists, duplicate questions aren’t allowed!")
       }
     )
   }

peer-prep-fe/src/edit-page/edit-page.component.html

@@ -87,6 +87,6 @@ <h2>CATEGORY</h2>
   </div>
 
   <div class="saving-button">
-    <button class="save-button" (click)="saveQuestion()">SAVE</button>
+    <button class="save-button" (click)="saveQuestion()" [disabled]="questionForm.invalid">SAVE</button>
   </div>
 </div>

peer-prep-fe/src/edit-page/edit-page.component.ts

@@ -117,7 +117,7 @@ export class EditPageComponent implements OnInit {
           this.onEditComplete()
         },
         (error) => {
-          alert("Error updating question")
+          alert("Question with the same title already exists, duplicate questions aren’t allowed!")
         }
       )
   }