search

dillongoh / pe

0 stars 0 forks source link

Unexpected time to autofill #8

Open dillongoh opened 1 year ago

dillongoh commented 1 year ago

Entering the command "ap n/test6 d/15-04-2023" gives me the following output:

image.png

The program has added a deadline timing of 3am which does not make sense to me as the user. Even if a time was to be displayed, I would expect midnight to be the default time.

nus-se-script commented 1 year ago

Team's Response

The current expected behaviour is that the app parses all deadline inputs into a date-time output due to the library used to parse natural language/date formats. If no time is provided by the user, the current time is taken as the time to use.

Due to the usage of a library, it would take a lot more effort to remove the time of the output if no time is provided by the user. The user can also edit the deadline to have a 12am time if needed.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: The team stated that the expected behavior if no time is provided is to use the current time. I did not test their app at 3am. I don't see this bug as NotInScope as this would clearly affect normal usage of the app.

## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** This is not an incomplete or missing feature but an unexpected behavior with their existing feature that they claim to have. (Storing of date and time for deadlines)