search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Editing of JSON file allows for multiple of the same field #2497

Open nus-se-script opened 7 months ago

nus-se-script commented 7 months ago

image.png

The user is allowed to specify multiple of the same field (name in this case in the JSON file and only the latest one is taken.

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

officialchengyud commented 6 months ago

Team's Response

Thank you for that.

As we mentioned in the UG, editing the JSON file is at your own risk and veering off from the format that MediCLI allows could result in unwanted outcomes. MediCLI does not crash if you add the same field, thus the program is still working as intended. In future editions, we look to implement JSON validation so these changes are checked by MediCLI. However, in this version it is not important enough yet.

Therefore, we believe this is a NotInScope issue.

Duplicate status (if any):

--