search

tandeshao / pe

0 stars 0 forks source link

No delete cascade effect. #12

Open tandeshao opened 2 years ago

tandeshao commented 2 years ago

Step to reproduce:

  1. Type assign -s 1 -l 1 to assign the first student to the first lesson.

  2. Type rmstudent 1 and the image below will be shown:

image.png

Expected: Lesson 1 to be deleted as there are no students currently attending the lesson after deleting student 1.

Actual: Lesson 1 still remains in the app as shown in the image below.

image.png

nus-pe-script commented 2 years ago

Team's Response

The deletion of a lesson should be explicitly done by the user. In the case where the tutor has specific time slots for his lessons, he would not want the lesson to be deleted even when there are no students attending the lesson. Besides, he will also be able to know if he has to teach the lesson when he sees that there are no students enrolled in that lesson.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]