nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Appointments cannot be scheduled for the past #1421

Open nus-pe-bot opened 6 months ago

nus-pe-bot commented 6 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.


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

officialchengyud commented 6 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

Duplicate status (if any):

--