search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Incomplete UML class diagram for the storage component #2984

Open nus-se-script opened 2 days ago

nus-se-script commented 2 days ago

There is no mention of linkedPersonsEntries in the UML class diagram for Storage which is a crucial part of your implementation. As a Developer, I have no idea how the link between a Person and a Event is stored when this is one of the biggest component of your application.

UML Diagram: image.png

addressbook.json:

image.png

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

antgyf commented 1 day ago

Team's Response

This is indeed something that we could have included to aid reader comprehension, and we will take note of it. However, the link and unlink commands are only a small component of the features that we have added, not one of the major features. The biggest component of our application is the ability to track events, as well as their schedules. Additionally, the omission does not disrupt functionality; it reduces clarity in the documentation.

Duplicate status (if any):

--