nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

ADD Feature does not validate the age of the patients appropriately #5288

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

Steps to reproduce:

Expected:

Actual:

Screenshots: image.png

This is problematic as this might be a typo on the part of the user and the lack of validation check on such input may reduce usability. You can consider supporting this for better convenience and usability for your users. One way you can do is by throwing a warning when such unrealistic age is being inputted.


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

jmsandiegoo commented 4 months ago

Team's Response

Though it might seem absurd regarding records that might have large value of age. Our team has actually allowed such behaviour as we aim for the user to have the flexibility to add medical records from the past as well this might be in use cases where clinics that might have been operating for a long time to migrate into our application. Though as stated in the expected section of the issue we it would definitely help to add some warning message and we have decided that this is something that can be pushed back to future releases. Thus, marking this as NotInScope.

Duplicate status (if any):

--