nus-cs2113-AY2122S1 / pe-dev-response

0 stars 0 forks source link

add workout command and the timing and date #260

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years ago

The schedule feature works well, but when you omit the timing, the program will immediately input the current time as the time recorded though the date may be in the past. It may not be the best when the date is already specified as in the past because it does not really make sense to put a previous date and the current time for recording purposes.

There should be an error message instead to tell the user to input the date that they exercised if the date was in the past. Screenshot 2021-11-12 at 4.43.08 PM.png


[original: nus-cs2113-AY2122S1/pe-interim#282] [original labels: severity.Low type.FeatureFlaw]

arvejw commented 3 years ago

Team's Response

There appears to be a misunderstanding about add workout. add workout records workouts and the calories burned doing (completed/past ones in conventional usage, but users can record in advance if they want). It gives users the ability to record entries in the past due to possibility of forgetfulness.

Scheduling of workouts is done by add schedule and is future-based, where workouts are assumed to be scheduled ahead of time in the future. add schedule will not retain workouts scheduled in the past and will delete/reschedule them depending on wheheter they are recurring workouts or not.

Duplicate status (if any):

--