search

JustinSoh / pe

0 stars 0 forks source link

Should not be able to add modules that requires pre-req before I have done the pre-req #3

Open JustinSoh opened 5 months ago

JustinSoh commented 5 months ago

Description: I should not be able to add a 2k module before having done the 1k module. E.g. adding in CS1101s after adding in CS2040C

Steps to Reproduce:

  1. Launch the app
  2. `add c\CS1101s w\4
  3. add c\CS2040C w\1

Expected Result: It should have prompted me with an error telling me that CS2040C requires CS1101s in order for me to do it. However there was no error. Perhaps there could be a warning telling me that I am adding a 2K mod and there is pre-req for it

Actual Result: Allows me to add it in.

Screenshot:

image.png

nus-pe-script commented 5 months ago

Team's Response

These requirements are varied and depend very much on the particular modules you are taking. This means that we not only need to have the database access to every single modules in NUS but also the specific information in the module. Hence, it is more feasible if the choice is left to the user to make. It has also been mentioned that the users have to be aware of the prerequisites of the modules before adding them as seen in the screenshot below.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.High`] - [x] I disagree **Reason for disagreement:** I agree that the issue is Not In Scope. However, I do not agree that the issue's severity is Low. The value proposition of the app is for `CEG students that have a lot of modules` and to make it such that `user can plan the modules they want to take in the future`. With pre-requisites being such a big part of module planning, and with a huge load of modules to manage - I believe this issue severity should be at least a medium as it doesn't make sense for users to do prior planning before using your planning application. In conclusion, considering that this limitation has been stated in the User Guide - I would change my Issue Severity to `Medium` as I do believe it is an important feature when it comes to the planning of modules.