nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Appointments have no minimum duration #3472

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

image.png

Currently, appointments do not have a minimum length, allowing users to set appointments as close as 1 min after each other. This makes it prone to user error as users who mistyped will setup an appointment which is unfeasible

Recommendation:


[original: nus-cs2103-AY2425S1/pe-interim#2651] [original labels: type.FeatureFlaw severity.Low]

bmanara commented 1 week ago

Team's Response

Our decision to not include end time/duration for appointments considered the unpredictable nature of the duration of appointments.

Example: appointment ends early and receptionist want to schedule a walk in appointment -> need to edit the end time of the first appointment, otherwise there would be a time clash when adding the new walk in appointment. This would end up creating more work for the receptionist.

Hence it would be easier to just add the new appointment first with a reasonable start time after any previously scheduled appointments (e.g 1h).

Due to this current workaround, we believe the current app can still function normally as intended and that the issue is out of scope for this iteration.

Duplicate status (if any):

Duplicate of #1077