search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Incorrect Storage Diagram [DeveloperGuide] #729

Open nus-pe-bot opened 5 months ago

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

There are classes that do not exist like JsonAdaptedPerson and missing classes like JsonAdaptedAppointment. This diagram is incomplete too as it not representative of how JsonAdaptedAppointment is related to the other classes used in storage Screenshot 2024-04-19 at 5.22.17 PM.png

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

vannykin commented 5 months ago

Team's Response

The appointment feature is not relevant to this diagram as this diagram was meant to be a brief overview of the storage component.

We also left out all other fields of a client as it did not add value to the purpose of this diagram.

Note: there are 2 bugs in this feedback, we only addressed one as required.

Duplicate status (if any):

--