search

liviamil / pe

0 stars 0 forks source link

MC not restrictive enough #4

Open liviamil opened 2 years ago

liviamil commented 2 years ago

image.png

The restriction for MC being from 0 - 45 is not reflective to the situation in real life, especially since the app is focused on NUS. More restriction was expected (i.e. max MC in NUS, e.g. 5 MC)

nus-se-bot commented 2 years ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

image.png

image.png

There are modules with 20mcs and even modules with more than 40mcs that we could find on NUSMODs.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]