nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Case Insensitivity on severity and sex #1926

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

Steps to reproduce:

add n/John Doehhhhhhdshdhdp/ilivehere e/johnd@example.com a/311, Clementi Ave2, #02-25 b/25/2/2024 s/female

expected able to add female

actual: unable to add due to female not "Female"

I feel this is against the module requirment of not too much input validation after all why should case sensitivity matter when the user is trying to specify the gender or severity when the intentions of the users is very clear


[original: nus-cs2103-AY2324S2/pe-interim#1598] [original labels: severity.Low type.FeatureFlaw]

jskimdev commented 4 months ago

Team's Response

We agree with this report. Thank you for the report. We have not taken into account the module requirement and forgot to change it in the later iteration.

Duplicate status (if any):

--