nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Proposed feature is still referring to addressbook in DG #6010

Open nus-se-script opened 1 year ago

nus-se-script commented 1 year 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.

This whole section is still based on addressbook, which is apparently a bug

_{[original: nus-cs2103-AY2324S1/pe-interim#5994] [original labels: type.DocumentationBug severity.VeryLow]}

Nixx162 commented 1 year ago

Team's Response

Since our application and the original AB3 share a lot of common groundwork, it is perfectly OK for the section to still be there.

Apart from the unimplemented classes that support the proposed feature, all classes mentioned in the section are implemented both in AB3 and in our application (no rename refactors were made to preexisting classes).

Of course, the proposed feature regarding undo and redo still remains a perfectly valid feature to be implemented in our project.

Duplicate status (if any):

--