search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

inconsistency in the way case sensitivity is handled #1343

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

Inconsistency in the way the application handles MAJOR_NAME for init function it only accepts uppercase e.g. CEG not ceg but for display function it accepts both upper and lower case e.g. CEG and ceg which can be confusing for users based on the UG which states that both display and init uses the same MAJOR_NAME

image.png

image.png

image.png image.png

[original: nus-cs2113-AY2324S2/pe-interim#1201] [original labels: severity.Low type.FunctionalityBug]

Hws2209 commented 4 months ago

Team's Response

This is actually caused by the fact that the app requires exactly the same case as what was specified in the UG for the MAJOR_NAME the first time each of the majors are used. It is not a difference between init and display.

This issue is a duplicate of another issue.

Duplicate status (if any):

Duplicate of #1344