SherniceSng / pe

0 stars 0 forks source link

Missing step in use case: delete a student #11

Open SherniceSng opened 1 week ago

SherniceSng commented 1 week ago

image.png

There should be an additional step after step 3a1, which specifies the tuition teacher correcting the mistake, else it will fall into error again.

nus-pe-bot commented 1 week ago

Team's Response

Already stated use case resumes at step 3.

So if teacher keys in an invalid input -> EduManage shows error -> Teacher keys in an input again

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I still think that it is important to state the extra step, especially due to the way your app is structured, where the user just needs to edit the index and not request the delete again. I do not think that the 2 actions (user just retype the index in the command again vs user request a whole delete command) are considered to be the same. Without specifying the step, this could make the use case inaccurate. Moreover, including the extra step aligns with the content given in the course website as well.

Screenshot of course website reference: image.png


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]