search

nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Insufficient staff error #1992

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Error thrown for new appoint command : newappt -id HELLOHELL -start 16/11/19 1200 -reoccur w -num 1000 eventhough i added a new doctor newdoctor -id 123456789 -name test who should be free.

Screenshot 2019-11-15 at 5.04.08 PM.png

[original: nus-cs2103-AY1920S1/pe-interim#2001]

woon17 commented 4 years ago

Team's Response

Thank you for your issue report. However, the result message displayed is correct based on the explanation in the UG.

image.png

As mentioned in the UG, appointments can only be scheduled at timings when there are sufficient staffing. Executing the command "newdoctor -id 123456789 -name test" only registers the doctor to the system, however, it does not imply that the doctor is on-duty all the time.

A follow up action is required indicate when the doctor is on-duty. Hence, the following shift management commands are provided:

image.png

Once a doctor is scheduled, such that there are more on-duty doctors than existing scheduled appointments, the user will then be allowed to schedule additional appointments at that timing.

[NOTE] Additional, the error message that you have reported does not tally with the example command that you provided newappt -id HELLOHELL -start 16/11/19 1200 -reoccur w -num 1000. The error message indicates that you attempted to schedule the patient's appointment from the timing of 2300 to 2330. However, in the example command you have provided it mentions scheduling reoccuring weekly appointments from 1200 to 1230. We understand that this mistake may be attributed to the time constrain of the PE.

Duplicate status (if any):

--