nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Duration for an internship does not allow leading zeros #3277

Open nus-se-script opened 10 months ago

nus-se-script commented 10 months ago

The duration of an internship does not accept an input with leading zeros, even though the number of months is a valid duration like 3 months. Moreover, the error message shown does not indicate to the user that leading zeros are not allowed, which means the user must figure it out on his own.

To recreate:

Results:

image.png

Perhaps a future iteration of the application can implement an error message to indicate this to the user, or ignore leading zeros.


[original: nus-cs2103-AY2324S1/pe-interim#2564] [original labels: severity.Low type.FunctionalityBug]

yongning0310 commented 10 months ago

Team's Response

Thanks for pointing out. I think it's a good feature implementation that we can consider. However, we also indicated that duration can only be a positive integer, and '03' is not a positive integer.

Duplicate status (if any):

--