nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

JSON Format Being Corrupted #315

Open nus-pe-bot opened 6 months ago

nus-pe-bot commented 6 months ago

Steps to reproduce:

Actual Output: Step 1: The user went to JSON file and change the field name of "tags" to something else and save it. image.png

Step 2: The user went to run the system as per normal on the terminal. image.png

Expected Output: It should be invalidated and displayed an empty list instead.

One example on how it should be validated. Step 1: The user went to JSON file and change the field name of "name" to something else and save it. image.png

Step 2: The user went to run the system as per normal on the terminal. As you can see here, the data is invalidated and the displayed list is empty image.png

Suggestion: Add validation to check for the field name "tags" to be validated.


[original: nus-cs2103-AY2324S2/pe-interim#417] [original labels: type.FunctionalityBug severity.Medium]

ys112 commented 6 months ago

Team's Response

Thanks for bringing up the issue. As per the user guide, editting data file will cause unexpected behaviour and this is expected. So, to further validate such error, it could be a future enhancement. And user would rarely edit it unless they are advanced user as specified by the user guide.

image.png

Duplicate status (if any):

--