search

nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Can add empty diary entries without error #2176

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

You can add empty diary entries. Maybe should throw error? (UG doesn't say anything about this)

Screen Shot 2020-04-17 at 2.40.12 PM.png

[original: nus-cs2103-AY1920S2/pe-interim#2137]

WANG-Yuchen-Alice commented 4 years ago

Team's Response

Dear tester, thank you very much for this report. We have decided to reject this issue as a feature flaw based on the following reasons:

  1. "What tester treated as a bug is in fact the expected behavior." In this case, building up a diary on a specific date with empty content is not an unexpected result based on the description in UG.
  2. This does not affect the operation of the product, as the user can still add content to this diary log. Below is a case when the user create an empty diary log and then add content to the diary.
  3. Stoarge wise, it is not harmful as when FitHelper is restarted next time, empty diary logs will be discarded automatically. Based on your command: call appendDiary d/2020-03-31 dc/Keep fit with FitHelper today! with the following effect:

image.png

Based on these reasons we reject this featureflaw report. Hope you can understand. We appreciate all your efforts.

Duplicate status (if any):

--