nichee / pe

0 stars 0 forks source link

Appointments cannot be scheduled for the past #10

Open nichee opened 4 months ago

nichee commented 4 months ago

image.png

This could be an issue for example if there is a walk in and a doctor wants to schedule an appointment to track that it happened

Also if they accidentally delete an old appointment, it cannot be added back

image.png

Even though the UG mentions this is an intended feature, this also applies just one minute after the current time, which can be inconvenient for the doctor.

nus-se-bot commented 4 months ago

Team's Response

Thank you for raising this issue. Indeed, we have intentionally made it so that appointments cannot be scheduled in the past since in the real world, hospital clerks would be scheduling appointments for the future (not doctors having an appointment first then putting it into the system - like what you said). This is reflected in our User Guide very clearly (screenshotted below).

Since its clearly defined in our User Guide, we believe this is a low severity issue rather than a medium. The MediCLI app is working exactly how we expected and designed for it to work as defined by the UG.

However, we did also think a little more about what you said and think that we could potentially add in a feature enhancement in the future that allows you to undo an appointment deletion. Thus, we decided this is a NotInScope issue.

Screenshot:

Screenshot 2024-04-20 at 3.46.14 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** Hi, I feel that the reasoning that the behaviour has been described in the User Guide should not have much bearing on how we evaluate this potential feature flaw. ![image.png](https://raw.githubusercontent.com/nichee/pe/main/files/6ae1e67d-0880-4a35-9cdf-b97f344cef85.png) "The MediCLI app is working exactly how we expected and designed for it to work as defined by the UG." has no bearing on a feature flaw, this statement only supports that there are no functionality flaws. I understand the reasoning as to why past dates are restricted - "hospital clerks would be scheduling appointments for the future" However the system allows you to make an appointment just 1 minute in the future (i.e. a walk in appointment). Not allowing appointments in the past simply makes the process of scheduling a walk in appointment on the app more inconvenient. We can not control how our users use our app. As someone who has worked in a medical centre and multiple hospitals, the vast majority of cases are walk in, not appointment based. As stated in the UG, an intended feature is to "track a patient/doctor's past appointments". This goal cannot be achieved if the vast majority of cases (walk-in) cannot be retroactively added.