cleoncheng2000 / pe

0 stars 0 forks source link

Edit priority repetition #10

Open cleoncheng2000 opened 1 year ago

cleoncheng2000 commented 1 year ago

image.png

tasks with priority that are already 0 can still be have their priorities deleted and set to 0 again

nus-se-script commented 1 year ago

Team's Response

We have already specified in the UG that certain commands with no effect can still be executed, but we may improve this in the future by displaying an error message instead.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Dev team mentioned that they may throw an error in the future meaning this should at least be in NotInScope and not outright rejected. Additionally, they should protect such commands from being executed as it does nothing with the elements in the todo list and is of no use to a user, instead the commands still execute and this is unnecessary computation.