search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Inconsistency in diagrams in DG #3211

Open nus-pe-bot opened 1 week ago

nus-pe-bot commented 1 week ago

image.png

In the above diagram, the multiplicities are stated wherever possible.

In the below diagram, multiplicities are stated for some cases such as a Person being able to have any number of Tag, but the multiplicities aren't shown for the associations from UniquePersonList to Person and from Person to other objects such as Name, Phone.

image.png

If the multiplicites for Tag is being shown, I think the others should be shown too.

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

lithops-zty commented 1 week ago

Team's Response

Thanks for pointing it out, but this is not in scope because:

  1. Multiplicity is optional
  2. Multiplicity of the other fields is not the main focus of that diagram (which shows a better implementation for the tag)

Duplicate status (if any):

--