kevinkuo0320 / pe

0 stars 0 forks source link

Can add non-alphabetic/numeric data into patient medical record. #5

Open kevinkuo0320 opened 2 years ago

kevinkuo0320 commented 2 years ago

As shown in the picture, there is non-alphabetic/numeric character stored in the height of the patient

image.png

nus-pe-bot commented 2 years ago

Team's Response

The input fields allow any values to account for different special values the user might input. This is made clear in the medical information section of the UG, in line with the recommendation on avoiding overzealous input validation.

Screenshot 2022-04-18 at 1.42.38 PM.png

Screenshot 2022-04-18 at 1.43.44 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Although it is stated inside the UG, it would be required to limit the invalid input to avoid misleading information stored in the database.


:question: Issue severity

Team chose [severity.Low] Originally [severity.High]

Reason for disagreement: [replace this with your explanation]