nus-cs2103-AY2223S2 / pe-dev-response

1 stars 0 forks source link

Bug with creating Medical Certificates - Unable to create MC without optional field of Medical Condition #867

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Steps to reproduce:

  1. Type "create 1 doc/Dr Tan d/1"
  2. Press enter to execute command

Expected: Creates a new Medical Certificate for patient at index 1 issued by Dr Tan for 1 day.

Actual: Error is thrown for incorrect command format and requests for the optional Medical Condition field to be provided.

Screenshot:

Screenshot 2023-04-14 at 2.30.48 PM.png Screenshot 2023-04-14 at 2.34.49 PM.png

As stated in the UG, the command for the input into the command line should be of the following format:

"create INDEX doc/DOCTOR_NAME [m/MEDICAL_CONDITION] d/DAYS"

It also states explicitly that the Medical Condition field is an optional field.

However, when attempting to create an MC for an existing patient without the optional field, an error is being thrown.


[original: nus-cs2103-AY2223S2/pe-interim#727] [original labels: severity.Medium type.FeatureFlaw]

WindyCall commented 1 year ago

Team's Response

Noted with the following, but I believe it should be severity. Low, as this flaw is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only. The first time when the user uses the application with CLI. With Error message, user should be able to handle.

Duplicate status (if any):

--