AayushMathur7 / pe

0 stars 0 forks source link

Incorrect error message for adding an appointment command. #2

Open AayushMathur7 opened 3 years ago

AayushMathur7 commented 3 years ago

image.png

When an appointment is added (in the patient/staff management control - not in the scheduler), there shouldn't be any error saying that the phone number has too few digits as there is no phone number in the argument. Maybe the error message can be changed asking user to switch to scheduler.

nus-pe-script commented 2 years ago

Team's Response

The commands for patient/staff management view are different from that of the commands in the appointment management view. Therefore, it does not make sense that a command intended for the appointment management view, but issued in the patient/staff management view, results in the error message that the reviewer has mentioned.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]