Quasant / pe

0 stars 0 forks source link

Commands require the n/NAME parameter, and if there are duplicates, the usage of "sn". This can be improved. #10

Open Quasant opened 1 week ago

Quasant commented 1 week ago

There are several commands where if duplicate student names exist, the user has to input the name and the student number.

This is an inconvenience to users since if name was not useful as an identifier, why not simply use student number from the start (which is guaranteed to be unique)?

Some examples include markat or adda.

Severity low as the odds of same name is low.

Examples include the below, where John Doe is repeated twice.

image.png

nus-se-bot commented 1 week ago

Team's Response

We have two interpretations of your suggestion.

  1. We use student number only - there is no way to delete by name. We believe this is objectively a downgrade to our current model since names are more intuitive for tutors. The student number is simply a way to disambiguate in the rare case of duplicate student names.

  2. We add another option to delete by student number only. This is a possible improvement in the future.

    Items for the Tester to Verify

    :question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]