nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Half bake detection for module code #1895

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Even though it is stated that module codes have to be capitalised, it would be better if it is able to take in lower case alphabets as well

Command: edit 1 c/Cs1101S

Expected: Module updated

Result: Error message

image.png


[original: nus-cs2103-AY2122S1/pe-interim#1953] [original labels: type.FeatureFlaw severity.Medium]

WuaaAj commented 3 years ago

Team's Response

NUS module codes are all completely capitalised. Therefore we followed this format to only accept capitalised module codes. It also standardises the look of all the modules in the application

Duplicate status (if any):

--