search

nus-cs2113-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Inproper response when viewing weight-record #674

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

When I delete the weight record first(then there is no weight record in the list), and when I want to view the weight progress, a message indicating invalid index comes out. It seems weird here since check-weight-progress does not require a index as parameter. I guess the warning message here should look like "currently there is no weight record" image.png

[original: nus-cs2113-AY1920S2/pe-interim#676]

yuxianglim commented 4 years ago

Team's Response

The error message means that there are no weight stored in the list currrently. Thus there is no weight-progress to be shown which fits the message that should be shown for the command too. We can add a few more lines of message if required in the future. This only occurs when user delete all the weight in the list which should not be the case as a person will always have a weight. We did not want to implement having atleast one weight stored as we want to make it flexible for user to delete any weight they wish to easily.

Duplicate status (if any):

--