search

lebencwb / pe

0 stars 0 forks source link

Inadequate identifying attributes of assigned instructors #4

Open lebencwb opened 3 years ago

lebencwb commented 3 years ago

Currently, instructors are assigned to the module by index and module code, whereas modules are searched by the name of the instructor. In the case where more than one instructors have the same name, when the user searches for modules by a certain instructor, they are unable to tell between the different instructors, when they might only be interested in finding modules of one of the instructors.

Steps to reproduce:

  1. add n/Janson Garrick p/98765432 e/jansongarrick@example.com d/Computer Science o/B01-A3
  2. add n/Janson Garrick p/98765431 e/jansongarrick1@example.com d/Computer Science o/B01-A3
  3. assign 7 m/CS1010
  4. assign 8 m/CS1231
  5. findmod i/Janson Garrick

image.png

It makes things more complicated when trying to unassign the instructor from the module as well:

  1. unassign 7 m/CS1231

image.png

nus-pe-bot commented 3 years ago

Team's Response

Thank you for your report! We have specified in both the UG and the DG this would be fixed in the upcoming updates. Since the fix is not essential for the app to be reasonably useful (e.g. very rare for people to have the exact same name), the bug should not be considered a feature flaw either and should only warrant a low severity if it is in fact a problem.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]