search

XylusChen / pe

0 stars 0 forks source link

Data file name is addressbook.json #3

Open XylusChen opened 1 year ago

XylusChen commented 1 year ago

image.png

As seen from image, data file name is still addressbook.json, which can be confusing for users since app is a list of patients.

nus-se-bot commented 1 year ago

Team's Response

Thank you for bringing up this issue. As you may recall, replacing "addressbook" with our own product name in the source code is optional in CS2103T.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: I don't think the team's response has addressed the issue. Yes the requirement is optional, but this does not change the fact that users may be confused when trying to manipulate the data file directly, which is supposed to be a feature of the application. The issue was raised not to fault the team for not replacing addressbook occurences in the project, but to show that it can cause hindrance to the user.