search

wujiaweijoanna / pe

0 stars 0 forks source link

Add command ALLERGY parameter #2

Open wujiaweijoanna opened 1 week ago

wujiaweijoanna commented 1 week ago

Based on the given examples, format for ALLERGY parameter should be [m/ALLERGY]... instead of m/ALLERGY...

屏幕截图(151).png

For e.g., when key in add command with m/peanut wheat, the displayed allergy is peanut wheat instead of peanut, wheat

屏幕截图(152).png

nus-pe-script commented 4 days ago

Team's Response

Based on the User Guide, to add in multiple parameters, there needs to be multiple m/ prefixes.

image.png

Furthermore, it is stated that square brackets denote that the parameter is optional. However, ALLERGIES is not optional as we believe it is important for users to acknowledge that the patient has no allergies to avoid confusion.

image.png

We have lowered the severity since users who have read the examples of the User Guide would be unlikely to make the same error.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]