search

kairoskoh / pe

0 stars 0 forks source link

Incorrect Index of Tasks #7

Open kairoskoh opened 2 years ago

kairoskoh commented 2 years ago

image.png

As seen in the image above, task 2 is "Do CS2113T tP". However, when finding the task, the task is labelled as 1.

This may become very confusing for the user and the indexes should be consistent.

nus-se-script commented 2 years ago

Team's Response

This is a design decision. The find command will list all the matching tasks. For example, if 3 matching tasks were found the output would be:

1. Do homework
2. Do tutorial
3. Do shopping

Note: simplification of program output

The output above is likely less confusing than the following:

3. Do homework
5. Do tutorial
8. Do shopping

Note: if actual task index was used instead

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]