nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Inaccuracy in DIAGNOSIS constraints #3571

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

Description:

In the UG, the constraints for DIAGNOSIS is "Must contain at least 1 alphabetic character and has a character limit of 80".

But within the product itself, empty diagnosis are allowed.

Steps to Reproduce:

For example, edit 1 d/ is allowed, despite the DIAGNOSIS not containing at least 1 alphabetic character.

Screenshot(s):

image.png

image.png Command was allowed.


[original: nus-cs2103-AY2425S1/pe-interim#2898] [original labels: severity.Low type.DocumentationBug]

kyouscrap commented 1 week ago

Team's Response

Thank you for your input. We missed out on that aspect as it is inherently optional.

Duplicate status (if any):

--