search

shawnnlimm / pe

0 stars 0 forks source link

d/DATE_TIME format not specified clearly / incorrect #9

Open shawnnlimm opened 5 months ago

shawnnlimm commented 5 months ago

Description

According to the UG, it seems that the d/DATE_TIME format only accepts [x]am-[y]pm, implying that an appointment can only start in the morning and end in the afternoon/evening. However, doing [x]pm-[y]pm] also works in the app.

Steps to reproduce

image.png

Expected Behaviour

UG should be clearer on the d/DATE_TIME format

Actual Behaviour

Missing details regarding d/DATE_TIME format

Label elaboration

Severity: VeryLow, since it's unlikely to affect usage

BugType: DocumentationBug

soc-se-bot commented 5 months ago

Team's Response

This is a fair point, and there was a tradeoff between an excessively long UG (there are three permutations: AM-AM, AM-PM, PM-PM) and a shorter, but more intuitive UG. We chose the latter option because we hoped that it was intuitive enough that scheduling morning-only or afternoon-only appointments is possible.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]