search

joohwan58 / pe

0 stars 0 forks source link

Command cannot handle multiple flags at once #1

Open joohwan58 opened 3 years ago

joohwan58 commented 3 years ago

The documentation gives me the impression that having multiple flags in 1 command would work, but it does not. Being able to record all the data for a patient in 1 line would help a fast typer get the work done faster.

image.png

image.png

nus-pe-bot commented 3 years ago

Team's Response

The correct command format should be /d allergy instead of d/ allergy. As shown in the screenshot, the command runs as expected when provided with the correct arguments.

image.png

In addition, we have also downgraded the severity to Low as it is not a "flaw that affects most users and causes major problems for users", as described in the module website.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]