LowTL / pe

0 stars 0 forks source link

[Timetable] Issue with adding long module codes #16

Open LowTL opened 2 months ago

LowTL commented 2 months ago

This is an actual course that I am taking. The timetable feature does not allow me to add this in.

image.png

soc-pe-bot commented 2 months ago

Team's Response

The current version of our software is designed to exclusively support 7-character module codes, which aligns with the standards and practices of our primary user base, which is NUS undergraduates, which does not include NUSC or graduate students.

In cases where an 8-character module code is attempted, our system does not crash but instead fails gracefully. It provides a clear and appropriate error message informing the user that the input does not match the expected format.

At this time, supporting 8-character module codes is considered out of the scope of our software. This decision is based on current priorities and the need to optimize support for the majority of our users. Future updates may revisit this functionality based on user feedback and changing needs.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: NUSC Students are also NUS undergraduates, so I think that your program should have been able to capture this.

Additionally, there are other NUS mods such as LSM2289R or ACC1701X that are not for NUSC nor graduates that have longer module codes.

Furthermore, in other modules within the same program (GPALIST, GPASTEP), I was allowed to input the same longer class code (NGT2001E) into the program.

The validation feels overly strict and insufficiently captures the needs of even your target audience.