evanyan13 / pe

0 stars 0 forks source link

[DATETIME] Unable to add in 2400 as an appointment datetime #5

Open evanyan13 opened 11 months ago

evanyan13 commented 11 months ago

Steps to reproduce:

  1. Enter the command edit n/Alex Yeoh ap/29-Feb-2024, 0000, 2400 and press Enter

Screenshot 2023-11-17 at 4.56.25 PM.png

Expected Result

Actual Result

Screenshot 2023-11-17 at 4.57.42 PM.png

Severity Check

nus-pe-script commented 11 months ago

Team's Response

Thank you for your feedback.

We understand that the idea of 2400 is referring to 0000 for the next day. However, as we have created this app for our target audience of the front-desk worker of small clinics which are unlikely to operate with appointments at that time of day, we believe that this is out of the scope of v1.4, as this is an extreme edge case for most clinics, and we had more pressing bugs to fix during v1.4

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]