search

termehlee / pe

0 stars 0 forks source link

/require does not parse time #10

Open termehlee opened 5 years ago

termehlee commented 5 years ago

The input is not the conventional time input (eg hours or mins) but still gets accepted

Screenshot 2019-11-15 at 4.49.47 PM.png

nus-pe-bot commented 5 years ago

Team's Response

The requires function gives the user flexibility to choose how long they want to do it for and the way they want to type it. They can type like 1 day or 1 hour or 1 min, instead of having the actual date or time. It is just how long they need to complete the task. So the time is a string, not a date and time.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

Team chose [severity.Low]. Originally [severity.Medium].

Reason for disagreement: [replace this with your reason]