DrWala / pe

0 stars 0 forks source link

DG contains references to AddressBook and not EzManage #19

Open DrWala opened 3 years ago

DrWala commented 3 years ago

What is the issue

DG still says that the Model "stores address book data"

What it should be

It should say "store EzManage data"

Why it matters

The developer reading this should not wonder where the "Address Book" came from

Severity justification

Low because it will cause a minor inconvenience to developers trying to understand the project, not VeryLow as its not purely a cosmetic issue.

nus-pe-bot commented 3 years ago

Team's Response

These are the actual class names in the backend code. Hence, we referred to the class data in the model explanation.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

[x] I disagree

Reason for disagreement: This was adapted from AB3 (well, not changed from AB3's DG), where I am certain they are referring to the product "Address book", and not the class. If it were a class, it would be written as AddressBook (code formatted).