search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Blank appointments are allowed #2278

Open nus-pe-bot opened 4 months ago

nus-pe-bot commented 4 months ago

Description Blank input appointments are allowed by the app

Steps to reproduce

  1. Launch the application for the first time to load sample data
  2. Run edit 1 o/

Expected behaviour The app should display an error message since no valid date was provided

Actual behaviour The app saves the empty input as an appointment Screenshot 2024-04-19 at 4.58.06 PM.png

Reason for severity Affects usage, blank appointments may be created with no way of tracking them

[original: nus-cs2103-AY2324S2/pe-interim#1872] [original labels: type.FunctionalityBug severity.Low]

jskimdev commented 4 months ago

Team's Response

This is intended behaviour, since there can be a case where the appointment is canceled and by giving empty input for an appointment, we wanted to allow the user to remove the appointment from the patient card. However, we have not stated this in the user guide, so could have caused a misunderstanding.

Duplicate status (if any):

--