nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

An edit that causes a duplicate appointment is not caught by the system #842

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

image.png

Given a situation where an appointment already exists, attempting to an appointment to be have the same details results in the system not catching the error and not prompting the user about the situation, which may lead the user to think that the system is not responding. Note the terminal throwing the error below. image.png


[original: nus-cs2103-AY2425S1/pe-interim#1183] [original labels: severity.High type.FunctionalityBug]

jayjay19630 commented 1 week ago

Team's Response

The GUI does prohibit duplicate appointments as mentioned in the UG, so it is working as intended. The issue is that the GUI is not telling the user that this is the case. Since this is an issue related to an incomplete terminal output and does not break/crash the application, we believe that the severity of this issue can be decreased to low.

Additionally, adding this terminal output was not as important as getting the edit appointment feature to work, so our team considers this as out of scope for now.

Duplicate status (if any):

--