nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Delete class feature implemented with name #5730

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Currently, to delete a class, the app uses the format deleteClass . Suppose the user adds a new class like : A012BXY6. To delete this class, they have to type out deleteClass A012BXY6 and this is not optimised for fast typing and can be inconvenient as the complexity of the class name increases.


[original: nus-cs2103-AY2122S1/pe-interim#5709] [original labels: type.FeatureFlaw severity.Medium]

Wilfredwongkc commented 2 years ago

Team's Response

Thanks for the issue. We reject this bug because we think it is the correct behaviour for “deleteClass” to require users to type out the entire name to prevent them from deleting a class (with all the students inside) carelessly. However, if you really want to consider this as a feature flaw, we think it is of low severity because this is unlikely to affect teachers’ normal use of the app and only appears in rare situations: 1) a teacher will not use the “deleteClass” frequently 2) under normal situations, teachers can be very familiar with class names, and it is not easy to type wrongly 3) if a user finds a class name is too long/strange to type out, he/she can always edit it to become a simpler one 4) under normal situations, it is illogical to have a class name that is that long and complex.

Duplicate status (if any):

--