search

Tanzhiheng26 / pe

0 stars 0 forks source link

UI Component Class Diagram #7

Open Tanzhiheng26 opened 2 months ago

Tanzhiheng26 commented 2 months ago

image.png

I believe the 1 should be near the tip of the arrowhead.

nus-se-bot commented 2 months ago

Team's Response

This seems to be an issue related to rendering the PUML diagram on GitHub page. That's why the PDF shows the same issue as the GitHub page. Thus, despite our actual diagram being correct, we could not do anything about it. image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: In that case, I think that there should at least be a note below the diagram pointing out this issue and clarifying the correct position of the 1.

## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** I don't think the bug is purely cosmetic because the 1 is meant to be the multiplicity of PersonDetailsPanel but it is displayed as the multiplicity of MainWindow.