BenFheng / pe

0 stars 0 forks source link

Edit course function does not work as intended #1

Open BenFheng opened 4 years ago

BenFheng commented 4 years ago

My course initially was computer science with 1 semester left, and i edited the semester and name to Computer Science and 4 semesters left. However it showed that i had 0 semesters left which should be a bug.

Screen Shot 2020-04-17 at 2.12.32 PM.png

nus-pe-bot commented 4 years ago

Team's Response

Semesters Left is automatically computed based on input of modules. So if changing total semesters to a number less than the original, it will update accordingly to a lower number. With total semesters of 4, modules with semester Y2S2 have semester count of 4 too, which results to semesters left as 0.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Seems to be a common bug that's actually a feature flaw. If I want to set that there are 4 semesters left, the app should respond accordingly. On one hand the app wants to provide flexibility to its users with the MCs completed, and another hand it wants to ensure that the number of semesters is reflected as intended, and is unable to be customised. I still think we need to set the number of semesters for ourselves, and not let the app calculate it for us.


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]