nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Remnants of AB3 in logfile and log statements #629

Open nus-se-script opened 6 months ago

nus-se-script commented 6 months ago

The logfile is still named addressbook.log, and the log statements inside reveal that the data file is still addressbook.json. This might mislead the user, as the data might get mistaken for the data file of another application, like AB-3, although this does not hinder the functionality of this app.

Screenshot 2024-04-19 at 4.18.45 PM.png


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

tanguoyang commented 6 months ago

Team's Response

Hi, we feel that this is not an issue because it does not affect the user experience of our product. Furthermore, the use of addressbook.json is also mentioned in our User Guide, so our choice of save file names should not confuse users.

image.png

Duplicate status (if any):

--