nus-cs2113-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Constraint-Human-Editable-File Violated #688

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

image.png

Application data is not stored in editable text file, going against project constraint


[original: nus-cs2113-AY2122S2/pe-interim#623] [original labels: type.FeatureFlaw severity.High]

darylhjd commented 2 years ago

Team's Response

Due to high linkage between different parts of the programme (for example, ensuring data integrity between orders and dishes), the team felt it was better to lockdown editing of the local data, even though this went against the project constraints.

ie. It would have done more harm than good to allow editing of the data.

In any case, this will not pose large occasional issues to the users of this application. They will rarely find the need to manually edit values in the data file. In the case where such a need arises, we believe it will be rare in nature and will not cause much of an inconvenience as compared to launching the application and editing the data from there.

For that reason, we believe the severity to also be Low.

Duplicate status (if any):

--