search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Application still uses AddressBookParser instead of using ClinicMate #4819

Open nus-pe-bot opened 2 months ago

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

Not all instances of AddressBook have been removed from the application and application still uses AddressBookParser and other AddressBook functions instead of it being ClinicMate

Screenshot 2024-04-19 at 17.17.20.png

[original: nus-cs2103-AY2324S2/pe-interim#4284] [original labels: type.DocumentationBug severity.VeryLow]

jiahui0309 commented 2 months ago

Team's Response

Hi, we have indeed AddressBookParser in our code, and we don't feel that it is incorrect to put the diagram in this way. We want to display the diagram exactly as how we implemented it in our code.

Thank you for your understanding :)

Screenshot 2024-04-20 at 10.43.36 PM.png

Duplicate status (if any):

--