search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Duplicate exams can be added #1988

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

If I key in "student -ae 1 s/Math d/2020-02-29 14:00" 3 times, 3 instances of the same exam are input into the student's exam list. There should be a check for duplicate entries!

Screenshot 2021-11-12 at 4.33.16 PM.png

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

tanhuakun commented 2 years ago

Team's Response

Downgraded: Medium -> Low.

While the bug is correct, it should not cause a lot of inconvenience to users. Exam dates are for the users to help their students score better, by noting the exams their students have upcoming.

Duplicate exams can be seen in the list, and not affect the user's usage of the app. For example, if the user notes that his student has an exam next Friday, it does not affect the user much as the user will be preparing that student for that exam anyway.

Also, as a user of the app, it is unlikely that a student has many exams that are of interest to the user. They may usually have 3-6 exams per semester that are related to the topics the user teaches. Looking through the list, a duplicate exam could be spotted quite easily.

However, it is definitely a good feature to include, hence it is accepted but downgraded in severity.

Duplicate status (if any):

--