search

SeeYangZhi / pe

0 stars 0 forks source link

Feature Flaw: Lack of support for adding students with multiple courses #10

Open SeeYangZhi opened 2 weeks ago

SeeYangZhi commented 2 weeks ago

It is not uncommon that a TA may teach more than one course, for example, CS1231S and CS2040S. Similarly, often students may attend more than one course.

Expected:

  1. Ability to add multiple courses through multiple c/ prefixes: add n/Betsy Crowe t/friend e/betsycrowe@example.com c/CS1231S p/1234567 t/struggling

Actual: Duplicate detected, unable to add students with multiple courses

Screenshot 2024-11-15 at 5.28.33 PM.png

nus-pe-bot commented 1 week ago

Team's Response

Hi, thanks for your feedback.

However, our application distinguishes entries based on the combination of email and courses. As such, TAs can input the same student from different courses as two different entries. This is intended as it helps with isolation between grades from different courses.

Therefore, since this is intended, we are rejecting this bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]