nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Wrong eligibility result for module #671

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

EE4410 needs EE3408C as prerequisite, and I have not taken EE3408C. However, the application says instead that I am eligible for EE4410 which is not right.

Screenshot 2021-11-12 at 5.02.45 PM.png

_{[original: nus-cs2113-AY2122S1/pe-interim#656] [original labels: type.FeatureFlaw severity.High]}

ChewyChair commented 2 years ago

Team's Response

Rejected

This is a really great catch! Unfortunately the issue here lies within the NUSMods API, and not our application. The data provided by the NUSMods API in this case happened to be incorrect. (Prereq is stated as E3408C instead of EE3408C.) Since E3408C is an invalid mod and does not exist on NUSMods, UniMods is unable to add it as a requirement and thus EE4410 does not have a module requirement.

Link to json https://api.nusmods.com/v2/2021-2022/modules/EE4410.json

Duplicate status (if any):

--