search

Arif-Khalid / pe

0 stars 0 forks source link

Adding spaces to time input to addExpense command results in wrong time format #5

Open Arif-Khalid opened 1 year ago

Arif-Khalid commented 1 year ago

I suggest warning the users about this specific error or accepting additional spaces since spaces are specified as the correct delimiter. image.png

nus-se-script commented 1 year ago

Team's Response

This is an intended behavior of our product. We did state that the format for time is 'YYYY MM DD HH mm'. So entering additional spaces between the numbers will cause an error as it is not following the format we stated. In our next iteration we can perhaps solve this problem by allowing multiple spaces between numbers. But for this version, we are standardizing the space between numbers to be one.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: By your very claim that this is something you can work on in your next iteration, you agree that this should be a behaviour that is worked on, thus, this should be response.notInScope over rejected.