nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

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

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

image.png 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.


[original: nus-cs2103-AY2425S1/pe-interim#3251] [original labels: type.FeatureFlaw severity.Low]

ghos7ie 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.

image.png

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

Duplicate status (if any):

--