G13nd0n / pe

0 stars 0 forks source link

Alphabets limitation in the add function #5

Open G13nd0n opened 2 weeks ago

G13nd0n commented 2 weeks ago

image.png

There seems to be a restriction in the type of course that can be inputted, since it is also possible for cs code to have alphabets

Steps to replicate: add cs2040de /pu unimelb /coursepu comp90086

nus-se-bot commented 2 weeks ago

Team's Response

Thank you for the feedback! However, the input cs2040de is not an approved NUS course mapping based on EduRec, hence it fails the course mapping checks with our University data, which leads to the expected output of the error message received.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

This is the user story provided, which states that the user can add in their course mapping for their partner university. Users can also seek for approval which is separated from the list of course provided from EduRec. Furthermore, EduRec only has pre-approved modules and users can still be able to request for alternate approval for their module mapping. Hence, but only limiting it to the EduRec approvals, it will cause inconvenience to the user when trying to add approval that was performed separately.