nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

app: Find student applies to all courses #1043

Open nus-se-script opened 10 months ago

nus-se-script commented 10 months ago

app: Find student applies to all courses

Affected Component(s)

[x] Application [ ] User Guide [ ] Developer Guide

Description

When finding a student from a course, say course A, the same search filter is also applied when I switch tabs to course B.

This behaviour is not stated in the UG.

Steps to reproduce

  1. In course A, find a student that exists in course A but not in course B
  2. switch tab to course B

Command(s) entered:

find n/bad

Expected behavior

When I switch tabs, I should see all students.

Actual behavior

I see no students as no students in the current tab matches the search filter

Screenshots, if any

image.png

image.png

Additional context / Potentially problematic behavior / Improvements

[x] I have checked that this has not been addressed in the Planned Enhancements of the Developer Guide.

Reason for choosing severity level:


[original: nus-cs2103-AY2324S1/pe-interim#1606] [original labels: type.FeatureFlaw severity.Medium]

anthonytamzil commented 10 months ago

Team's Response

This issue of the find student applying to all courses only would cause minor inconvenience. Ultimately, there would be no actual loss of data and there is a viable workaround it by removing the filter and listing all students. Furthermore, this issue does not persist if the course switch command is used, which is the intended action, and only occurs when the user changes address books using the GUI by clicking on other courses, narrowing the scope of the problem. However, this issue is accepted as a potential feature flaw that could be improved or better documented down the development road.

Duplicate status (if any):

--