search

clyveycui / pe

0 stars 0 forks source link

Error message for interview time. #6

Open clyveycui opened 3 years ago

clyveycui commented 3 years ago

One of the example given in the error message doesn't work, as

"2021-9-1,08:00"

nus-pe-bot commented 3 years ago

Team's Response

image.png

This is a typo in the UI. As seen in the picture, we have another example that has the whitespace included.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: While it certainly can be seen as a cosmetic issue, the main reason why I placed a low severity instead of a very low is because this issue is with one of your examples in the error message, rendering it invalid for the actual command. This may cause a lot of confusion for users who are still not too comfortable with the date time format used here.