nathanshew / pe

0 stars 0 forks source link

Lacks Edit Appointment feature #15

Open nathanshew opened 1 week ago

nathanshew commented 1 week ago

image.png There is no edit appointment feature, requiring users to delete then add back appointments.

nus-pe-script commented 1 week ago

Team's Response

Appointments are designed to be immutable and we be believe the delete-add flow provides a viable alternative for users who need to make changes. Given this alternative, we feel that the severity of this issue should be Low instead of Medium.

Furthermore, we believe that adding an edit feature will require more effort that outweighs the marginal benefit to users, and as such will be considered not in scope:

image.png

Thank you!

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: I believe that the benefit is much more than a marginal one as suggested by the developer, due to the length of the add and delete commands. This requires the user to type much more characters.

addAppt Dental i/S1234567A @d/2024-10-27 @t/1100-1200 delAppt i/S1234567A @d/2024-10-27 @t/1100-1200


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** `addAppt Dental i/S1234567A @d/2024-10-27 @t/1100-1200` `delAppt i/S1234567A @d/2024-10-27 @t/1100-1200` Frequency of this is at least occasional, given the length of the add command, and hence tendency of errors Similarly, the inconvenience is not minor due to the length of commands.