gyant6 / pe

0 stars 0 forks source link

findTask cat/ not working as intended #4

Open gyant6 opened 4 years ago

gyant6 commented 4 years ago

In this first screenshot, I have tasks under the category "School" (added using moduleAdd) and tasks under the category "School work" (added using deadlineAdd).

image.png

If I enter the command findTask cat/School work, the app does not respond even if i change the entry to findTask cat/school work (matching the category on a letter-case basis)

Capture.PNG

If I enter the command findTask cat/School, only the tasks added under school work is listed. This means that i cannot filter my tasks with the category school even though it is added in a main functionality through moduleAdd

Capture.PNG

nus-pe-bot commented 4 years ago

Team's Response

The feature you mentioned maybe implemented for future releases. For this stage, the outcome shown is just the intended outcome which means if the user want to findTask by category, the user need to add in through deadlineAdd instead of moduleTask. If you add in through moduleTask then the user can use 'findTask m/MODULE_CODE'

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Mentioning that it might be for future releases seems like they are shying away from the fact that feature is still broken for this release, and it was also mentioned as a specific feature in the UG for this release. Their current implementation still does not allow users to differentiate between similarly named categories. Their rejection had no basis and they requested users to use other features (deadlineAdd) instead of the buggy / broken one (moduleAdd).


:question: Issue severity

Team chose [severity.Medium] Originally [severity.High]

Reason for disagreement: Users are not able to use both deadlineAdd and moduleAdd together. As both these features are important features in their application, them being buggy / broken takes away a large part of the functionality of the application. Severity should remain high.