search

Hazel1603 / pe

0 stars 0 forks source link

Storage not updated #28

Open Hazel1603 opened 3 years ago

Hazel1603 commented 3 years ago

Storage only shows person. Did not change to updated meeting and Modules.

nus-se-bot commented 3 years ago

Team's Response

The diagram is used to show a pattern used when designing the storage component rather than the entirety of the storage component. The addressbook is meant to be an example. But we understand that it may lead to a misunderstanding so we could have been more explicit.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low] Originally [severity.High]

Reason for disagreement: I would understand if the diagram were not updated to demonstrate a pattern. However, there should be some explanation of how your new entities (module and meetings) are stored, especially since meetingbook.json and modulebook.json has been created under data. There is no signs of that. Your omission of this information hinders the understanding of your application for many users.