search

nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Logic Class Diagram contains AddressBook #149

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

AddressBookParser is not edited to the parser name that the app is using. It might be better to refactor AddressBookParser to PartyPlanetParser as it is unclear what address book is or why address book is used image.png

[original: nus-cs2103-AY2021S2/pe-interim#149] [original labels: severity.Medium type.DocumentationBug]

nickyfoo commented 3 years ago

Team's Response

This is not a documentation flaw, as our parser was chosen to be called AddressBookParser, and this UML diagram is in line with our code. It is thus not in scope as it is not related to the quality of the documentation.

Duplicate status (if any):

--