search

nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Unrefactored part: undo/redo in DG. #2607

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 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.

Undo/redo part is not refactored from Ab3.

While the text is refactored, the diagram is still from AB3.

image.png

[original: nus-cs2103-AY1920S2/pe-interim#2607]

terencemah commented 4 years ago

Team's Response

This issue is not valid because we are still using the class AddressBook in our functional code. Thus, this depiction in the DG is still accurate and does not require refactoring. We also changed the severity to Very Low as this is only an issue of naming and does not affect in any way a developer's understanding of how the code works/interacts.

Duplicate status (if any):

--