ngoivanessa / pe

0 stars 0 forks source link

Training time input not checked #5

Open ngoivanessa opened 2 years ago

ngoivanessa commented 2 years ago

The time keyed in is not a valid time, yet it was accepted as an input

Screenshot 2021-11-12 at 4.45.57 PM.png

nus-pe-script commented 2 years ago

Team's Response

The '/a' portion is meant to be keyed in as a string. Thus, dates such as 'Friday PM' or 'Friday 1200' or '1200' are all accepted and meant to be allowed for the user to format himself.

This is also justified in the UG

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]