jiakai-17 / pe

0 stars 0 forks source link

app: Find student applies to all courses #7

Open jiakai-17 opened 8 months ago

jiakai-17 commented 8 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:

nus-pe-bot commented 8 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.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: Thank you for your response.

I believe that this issue would still cause occasional inconvenience to the user (when switching courses with the GUI), which is in line with the meaning of severity.Medium.