search

KrashKart / pe

0 stars 0 forks source link

Hard to view list of appointments for a doctor #7

Open KrashKart opened 1 week ago

KrashKart commented 1 week ago

Expected: I can expect to view the list of appointments for a doctor easily without scrolling due to Constraints: Typing preferred mentioned in the tP constraints.

Actual: I have to scroll down to view appointments if they exceed 3 for a certain doctor. image.png

Steps to reproduce:

  1. Assume John Tan and John Lim are doctors in the list
  2. Assume John Doe, John Teo and John hehe are patients in the list
  3. Enter these commands: a. add-appt pn/John Doe dn/John Tan d/23-04-2024 t/1100 b. add-appt pn/John Teo dn/John Tan d/23-04-2024 t/1200 c. add-appt pn/John hehe dn/John Tan d/23-04-2024 t/1300 d. add-appt pn/John Doe dn/John Tan d/23-04-2024 t/1400

Suggestions: Perhaps add another list users can switch to (eg view appt and view doctors, etc) so they are able to view all appointments and filter by doctor/patient

soc-pe-bot commented 1 week ago

Team's Response

Changing Severity from Medium to Low

As a team, we agree that this is a feature flaw since the user can only see the first 3 appointments and are forced to scroll through in order to see the other appointments for the person. However, we do not think that the severity is medium, as the Constraints: Typing Preferred states that typing is preferred for input only. Therefore, we interpreted that scrolling through the appointments list does not violate the constraint. (similar to how we scroll through the patient/doctor list)

Still, we understand that this is indeed a feature flaw and would want to implement a command to easily search the appointments of a person/date in a future iteration.

image.png

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: I seek to disagree.

The requirements for severity.Low are as follows: image.png

That is, the bug appears in rare situations and causes minor inconvenience only. However, it is stated in the UG and DG that: image.png "[MediContacts] is targeted at and built for receptionists working in small clinics, who need to manage thousands of patients, doctors, and appointments."

Thus, it is reasonable to assume that thousands of patients would incur many appointments, meaning that the problem of the receptionist having to scroll down the small appointment list would occur quite frequently, in line with severity.Medium: image.png

As for the constraint, I do agree that I may have misread the requirement, but I would also like to quote Recommendation-CLI-First: image.png That is to give a CLI alternative to mouse/GUI inputs if applicable.

If we consider "scrolling" as a "mouse input", no such CLI functionality was given to find appointments under a certain doctor, you would have to manually scroll down the list itself. This is also not addressed explicitly in the FAQ and Future Features in the UG or the Planned Enhancements in the DG.