search

sebin0817 / pe

0 stars 0 forks source link

Missing multiplicity for `Model.java` UML diagram #18

Open sebin0817 opened 2 months ago

sebin0817 commented 2 months ago

image.png

There should be 1 at the arrow heads near Name, Phone, Address, Email as these are compulsory fields to define a Person.

nus-pe-bot commented 2 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

Thank you for the bug report.

Our team has decided to accept it but have decided to lower the severity to very low as multiplicity is not required for UML class diagrams (according to the textbook) and should hence be a cosmetic issue.

image.png

However, given that we have added multiplicity for Meeting and there are multiplicities of 1 in our diagram, we decided that we should have added 1 to the respective fields for clarity as well.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your reason]