bryanyee33 / pe

0 stars 0 forks source link

Filter only allows tags #7

Open bryanyee33 opened 4 months ago

bryanyee33 commented 4 months ago

Filter only allows searching for tags, but for the use case for academic instructors, it is likely they would search by other fields such as name, id, major, modules, etc. Disallowing filtering by such would make it much harder for users to find specific individuals they are looking for (eg. searching for all students in CS2103T), especially if there are thousands of students.

nus-se-bot commented 4 months ago

Team's Response

Team chose [response.NotInScope]

Reason for disagreement: While I do agree that I have missed out the find command (why is this a separate command?), the 2nd point is not a justification.

The devs placed this issue under response.NotInScope, with the reasoning of "Major and Module can be done in future iterations.", even though there is no mention of fixing this issue in the UG or DG in the future. Also, there was no explanation on whether rectifying this is less important. Hence, their reasoning is invalid and insufficient, and there is no justification provided for putting it under response.NotInScope.

image.png


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.High`] - [x] I disagree **Reason for disagreement:** No explanation was provided for reducing the severity. This severity is justified because NUS academic instructors are likely to manage hundreds, if not thousands of students. Being unable to filter by a key component such as modules greatly takes away from the app's ability to "facilitate seamless management of academic contacts and data". For example, if the user wishes to search for all students in CS1101S in the app to find the median score, they would have to scroll through potentially thousands of students to find and sort through them. Moreover, filtering for students is likely the main purpose of the app, as there would be no point in adding all of these details of students and the grades for specific modules into the app otherwise.