search

flerovious / pe

0 stars 0 forks source link

Not possible to add certain mappings to uni #7

Open flerovious opened 2 years ago

flerovious commented 2 years ago

Screenshot 2021-11-12 at 5.12.38 PM.png

nus-pe-script commented 2 years ago

Team's Response

The help command as well as the User guide has clearly stated to use mapping index from searchmap command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The help command doesn't provide details on this. Perhaps this could be explained more clearly in the seachmap command in the user guide. Additionally, more helpful error messages could have been printed to the user for different error case scenarios. For example, I get the same error even when I'm trying to map a module that hasn't been added to my selected modules list yet as seen below where university with index 75 has already been selected.

Screenshot 2021-11-18 at 9.51.37 PM.png

:question: Issue severity

Team chose [severity.Low] Originally [severity.High]

Reason for disagreement: [replace this with your explanation]