search

nus-cs2113-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Command cannot handle multiple flags at once #88

Open nus-pe-bot opened 3 years ago

nus-pe-bot 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

[original: nus-cs2113-AY2021S2/pe-interim#88] [original labels: severity.High type.FeatureFlaw]

brandonfoong 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.

Duplicate status (if any):

--