search

AgentHagu / pe

0 stars 0 forks source link

Unneccesary difficulty in editting appointments #5

Open AgentHagu opened 4 days ago

AgentHagu commented 4 days ago

Description:

There's currently no easy way to directly edit a patient's appointment. This makes it tedious to edit something simple like the description of the appointment.

It's possible to makeappt for a patient with an existing appointment as a way to "edit" since it overrides but if the new appointment overlaps with the old one, this is not possible.

The only way to reliably edit an appointment is to first delete it then add the new appointment. But this is possibly tedious.

Suggested Improvements:

Add a new command like editappt which allows users to specifically edit the description, start or end date/times.

For e.g., editappt 1 a/New description allows users to simply edit the first appointment's description.

Benefits of Improvement:

Users can easily edit appointments (which is very possible for doctors and nurses to have to do), making the process easier and more intuitive.

soc-se-bot commented 18 hours ago

Team's Response

image.png

Thank you for your input. Because there is already a workaround regarding this issue, we do not feel like there was a need to work on this issue this iteration as other issues and features took higher priority.

The 'Original' Bug

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

Not allowing new appointment to overlap existing ones

While it is mentioned in the UG that new appointments replacing old ones are not allowed to overlap, it does not seem realistic to have this as a constraint. For example, if a doctor wants to reschedule an appointment from 10am-11am to 10.30am-11.30am, they will be unable to do so with the make appointment feature.

image.png

While this seems like a problem that may happen frequently, there is a workaround - deleting the appointment and adding a new one - so it can be considered a medium severity bug.

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

Their Response to the 'Original' Bug

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

image.png

Thank you for your input. As mentioned in the FAQ section in our User Guide, we have specified that we do not recommend editing an appointment via overwriting the current one. We suggest users to delete their existing appointment first. Because there is already a workaround regarding this issue, we do not feel like there was a need to work on this issue this iteration as other issues and features took higher priority.

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.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]