Open avellinwong01 opened 3 years ago
/d
and /t
takes in a date
and time
in the format dd/mm/yyyy
and hh:mm
, while /c
takes in calories burned as a positive integer. A such it can be inferred that the calories is the issue, since implementation wise, it is the first exception thrown.
That being said, the message could have been more intuitive, so this will be accepted.
Team chose [severity.VeryLow
]
Originally [severity.Low
]
Reason for disagreement: I believe that this unclear error message can potentially cause minor inconvenience and slightly confusion to the user. As such, I believe that it is not a purely cosmetic bug.
It is unclear which attribute a positive integer is supposed to be entered, since all attributes are left blank here.