fyshhh / pe

0 stars 0 forks source link

remind command needing all indices to not have reminders can be cumbersome #4

Open fyshhh opened 3 years ago

fyshhh commented 3 years ago

to replicate:

  1. set reminders of some assignments on.

  2. attempt to set reminders of multiple assignments with the remind command.

the command indicates which assignments have reminders set already, but this can be extremely cumbersome should i enter hundreds of indices as the argument to the remind command. does this mean i have to sift through what i typed to individually remove each index with a reminder set already? i think it would make more sense to add assignments without reminders to the list nonetheless and to display the same error message.

this is also apparent when using the done command.

Screenshot 2020-11-13 at 4.39.38 PM.png

nus-se-bot commented 3 years ago

Team's Response

Although it might be more convenient for the user if we set reminders for the assignments specified, we do not believe that the feature becomes less useful to the user. Given that the user entered some invalid indexes, there might be a chance that some of the remaining indexes were mistakenly added. Hence, we felt it was better to show the error message with the invalid indexes without setting reminders for any assignments.

Furthermore, as we included a list of the invalid indexes, we felt that it would not be cumbersome for the user.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]