search

jwdavis0200 / pe

0 stars 0 forks source link

Appointment not shown in HealthContact when trying to add an appointment. #7

Open jwdavis0200 opened 1 year ago

jwdavis0200 commented 1 year ago

This may be very confusing to users especially new users.

to reproduce: 1) add John as a patient then use aa feature to add appointment aa n/John s/2021-02-28 00:00 d/Dioni Wong t/x-ray. 1) slp 1 (Assume this person is another previoussly added patient with name adam not john), the appointment list is emptied since adam has no appointment. 2) enter aa n/John s/2021-02-28 00:00 d/Dioni Wong t/x-ray again.

Expected: show user all the appointments so he can check to see his list of appointments. Actual:

image.png

As you can see, there are no appointment listed, and the app warns of duplicate appointment, this may confuse new users(like me) when they forgot (or other potential cases) that they are in Adam's list(in this bug report example patient(refer to 2)).

nus-pe-bot commented 1 year ago

Team's Response

This is an intended functionality and hence not a bug. Adding a new appointment shoud not refresh the appointment list to show all appointments. Instead, if the select function is used before add, the screen should show the filteredappointments specified by the select command and after the add command, the screen should stay the same.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I disagree with the team's response strongly for this one. During the PE itself, while meddling with the features of the app, I was stuck in this state for quite a few minutes. At least the team should have provided an additional warning when in select person mode on top of the This appointment already exists in the HealthContact say for example You are in select person mode, the appointment already exists in the HealthContact, to view all appointments enter ... so that the user is not confused when he sees that there are no appointments listed in front of him and yet when he tries to add an appointment (that has already existed in the global list) the app prevents him only telling him This appointment already exists in the HealthContact (a very frustating experience that may stop him from using the product if I may add).

Thus, this indeed at least causes a minor inconvenience to the user when they start to use the app and hence is a functionality bug at the very least in the fact that the app should be telling the user that he may not be seeing all appointments since he is in select person mode.

While I understand the team saying " Adding a new appointment shoud not refresh the appointment list to show all appointments. Instead, if the select function is used before add, the screen should show the filteredappointments specified by the select command and after the add command, the screen should stay the same." I agree to an extent as well! If thats the implementation chosen by the team so be it, but what I am trying to convey is that the user should not be put in a position where he fails to add an appointment because the app warns him that the appointment already exists(and no additional information), although there are NO appointments displayed to him.