Deleteappt command is unnecessarily long

[dwsc37]

In the current system, each patient can only have 1 appointment. As such, it would be more convenient for the user if the command was just deleteappt INDEX or deleteappt NRIC.

[nus-pe-bot]

Team's Response

It is mentioned that in future iterations we will allow adding multiple appointments to a patient. Deleting appointment with date and time is therefore necessary to ensure the correct appointment is deleted.

Moreover, the hours and minutes are used as a fail-safe to ensure the end user does not delete the wrong appointment by accident.

For example, the receptionist may assume that the appointment for the 1st person is at 2pm and want to delete it because of a timing clash. Then he enters "deleteappt 1", the receptionist may delete an appointment for that person without knowing it is at 4pm instead and there is no timing clash.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

• [x] I disagree

Reason for disagreement: The features should be evaluated based on the current state of the app. It does not make sense to design a feature to be suboptimal for the current state of the app, in order to support a future planned enhancement. As it stands, current users trying to delete appointments are likely to face inconveniences each time they use the deleteappt command.

Additionally, I believe the scenario given by the team is unlikely, compared to the much more common scenario where the staff is just deleting appointments after the patient's appointment is over to clear the old appointment. In this more common scenario, each appointment being deleted will incur inconvenience to the user.

---