Jajared / pe

0 stars 0 forks source link

Missing attributes in diagram #13

Open Jajared opened 6 months ago

Jajared commented 6 months ago

Diagram can be more detailed to include attributes for policy such as policy number, premium, etc

Screenshot 2024-04-19 at 5.39.47 PM.png

soc-pe-bot commented 6 months ago

Team's Response

This diagram does not need to include attributes of its field, if not it would make the diagram very cluttered. As seen from AB3, the initial diagram did not include attributes details of Address, Email, Name , etc. Hence we did not feel the need to include it in this diagram.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: Screenshot 2024-04-23 at 4.52.04 PM.png

In fact, AB3 does include attributes of details of a Person, such as Address, Name, Email, etc. Because Policy is an object with multiple attributes to it, such attributes should also be included to be clearer. It is necessary to show the levels of abstraction. Adding one additional layer of attributes is unlikely to make the diagram cluttered.


## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** This can be classified as a Low severity instead, given that it is more than cosmetic issues but is unlikely to cause any significant issues.