search

Woolicious98 / pe

0 stars 0 forks source link

Storage link is confusing #16

Open Woolicious98 opened 3 years ago

Woolicious98 commented 3 years ago

Shown in overall:

image.png

Storage uses model.

In model component, the storage link is missing.

image.png

nus-se-bot commented 3 years ago

Team's Response

Omitting optional notations is not a bug as long it doesn't hinder understanding.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: It is understood if you omit optional notations in a higher level diagram such as main architecture. However in your case, main architecture display the notation but the more in-depth diagram does not.

In my opinion as to why this is confusing as the readers want to know what exactly does the Storage depends on in the Model component, it could be so many classes such as TestManager, DeckManager, Deck, AnswerList etc.

The explanation of the diagram also did not state any information of the Storage portion.

image.png