search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Changing the JSON file can lead to nonsensical scenarios #5853

Open nus-se-bot opened 3 years ago

nus-se-bot commented 3 years ago

image.png

It is valid for the user to set frequency and occurrence by changing the json file. This leads to nonsensical data such as this one.

Expected Behavior:

Count this as invalid data as it is not allowed to happen when adding by command

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

luayida99 commented 3 years ago

Team's Response

The issue is well noted, thanks for pointing it out. However, we think it is closer to a functionality bug since editing the json file constitutes functionality.

Duplicate status (if any):

--