vishnuvk47 / pe

0 stars 0 forks source link

Invalid error response #12

Open vishnuvk47 opened 1 year ago

vishnuvk47 commented 1 year ago

Screenshot 2023-04-14 at 5.07.57 PM.png

Allowed me to mark at a grade of "CS" for the mod when I used taken command, yet when I try to use mark and try grade of 's' it says I cannot su. this error should be prompted at the point of taken command itself

nus-se-script commented 1 year ago

Team's Response

Error response is valid as the module cs3244 cannot be SU-ed. Related to the CS/CU handling of the grade instead.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Does not check for whether a mod can be SU or if it is CS/CU

With access to the NUSMod API, perhaps there should be validation to check whether the modules are able to be SU-ed, or if they are CS/CU.

As shown below, CS2040, which is not a CS/CU mod can be marked as such, and CFG, which is a CS/CU mod, can be given a grade. This can lead to inaccurate reporting of CAP and planning, which is why I've marked it as medium

image.png


[original: nus-cs2113-AY2223S2/pe-interim#2231] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

CS/CU feature was proposed in the DG, and has not been fully implemented.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]