ChuanXinNg / pe

0 stars 0 forks source link

Fails to add appointment for different patients and doctors on a same timeslot. #2

Open ChuanXinNg opened 1 year ago

ChuanXinNg commented 1 year ago

Steps to reproduce:

  1. add appointment appt /for 1 /doc 2 /d Blood test /on 01-01-2024 13:00
  2. add appointment appt /for 3 /doc 1 /d Blood test /on 01-01-2024 13:00

image.png

Expected: Appointment can be added as these are different patients and doctor.

Actual: Cannot add appointment.

It is not reasonable that a clinic only has 9 timeslot per day for all patients and doctors. The timeslots should be based on the doctor, every doctor has a list of available timeslots.

nus-se-bot commented 12 months ago

Team's Response

This feature flaw has already been added into the planned enhancement section of the DG

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Incorrect output for view command

There are two doctors in clinic, and only one doctor has appointment at 9am, which means another doctor is avaliable at that time.

image.png

However, the output of view command shows that 9am is not avaliable.

image.png


[original: nus-cs2103-AY2324S1/pe-interim#5692] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

image.png

As currently our product is for small scale local GPs, we have designed our program to only have 1 room and hence 1 timeslot at any time. Only 1 doctor is working at a single time. This was also mentioned in our DG's planned enhancements hence we will reject this bug as mentioned in course website

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.Rejected`] - [x] I disagree **Reason for disagreement:** This should be not in scope as this would be an enhancement that the team needs to make for the future version of the app. ![image.png](https://raw.githubusercontent.com/ChuanXinNg/pe/main/files/ec372ca5-2fe8-45b9-8a48-966352affff5.png)
## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [x] I disagree **Reason for disagreement:** This should be a feature flaw, as this is not a functionality that does not work as specified or expected; it works as specified by the UG but should have been designed to work differently, which is categorized as feature flaw. ![image.png](https://raw.githubusercontent.com/ChuanXinNg/pe/main/files/56539ca6-c8b0-448d-b6a3-5bac753f0ca1.png)
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** This is not just a cosmetic problem, thus it should be seen as low or medium. Deciding this to be medium is because this has caused inconvenience to the users, as adding the same timings for a different patient and doctor will happen many times in a clinic. ![image.png](https://raw.githubusercontent.com/ChuanXinNg/pe/main/files/26121f6e-bb3d-4845-a24d-872ac2c8243d.png)