search

rachtan27 / pe

0 stars 0 forks source link

Able to add appointments that are "many days long" #4

Open rachtan27 opened 1 year ago

rachtan27 commented 1 year ago

I believe medical appointments would likely have a limit to the duration. It is illogical to be able to have an appointment that is 2 days long, because even if the patient is hospitalized, it does not count as an "appointment".

steps to reproduce: (assuming user exist)

add_appt n/Charlotte Oliveiro ts/06032023 10:00,08032023 11:00 d/Regular checkup doc/Xiao Lu

expected: error related to duration of appointment exceeding a certain limit

actual: command successful

screenshots:

image.png

nus-pe-bot commented 1 year ago

Team's Response

Hi,

There can be a case that the patient in question is hospitalized when a 2-day appointment makes sense. We believe that this is not a feature flaw since we give the clinics complete freedom to define what an appointment is as long as start time is before end time.

We thus reject this bug report.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: By not setting a limit to the length of the appointment and giving "clinics complete freedom to define what an appointment is as long as start time is before end time.", and hence allowing them to classify a 2day hospitalization as an appointment that they can save into medimeet, medimeet would be useless for the clinic in tracking other appointments related to other patients that are visiting the clinic during that 2 day hospitalization of 1 patient since their implementation prevents overlapping appointments from being added.