flyingsalsa / pe

0 stars 0 forks source link

Delete student requires exact student number rather than just index in the list. #5

Open flyingsalsa opened 1 week ago

flyingsalsa commented 1 week ago

It seems strange that while the edit command allows the use of INDEX as an identifier, the delete command does not and actually requires that you manually type out the entire student number. Remembering such a detail is not an ideal user scenario and also makes the commands seem inconsistent with each other.

soc-pe-bot commented 1 week ago

Team's Response

As mentioned in our DG, we intended for this to be the case since Delete Student is the more destructive process.

Undo/Redo was implemented after we made this decision, so we did not change the behvaiour.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

[ ] I disagree

Reason for disagreement: [replace this with your explanation]