Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mixed use of class diagram styles #12

Open
Dethada opened this issue Apr 19, 2024 · 1 comment
Open

Mixed use of class diagram styles #12

Dethada opened this issue Apr 19, 2024 · 1 comment

Comments

@Dethada
Copy link
Owner

Dethada commented Apr 19, 2024

The 2 different styles of class diagrams are both used in the DG, making it confusing to the reader.

image.png

image.png

@nus-se-script
Copy link

nus-se-script commented Apr 22, 2024

Team's Response

Thank you for raising this issue. However, our team has decided to showcase our ability to use multiple UML diagram types as recommended in the course website:

telegram-cloud-photo-size-5-6138422720812140293-x.jpg

Furthermore, this does not affect the usage of the app. Hence, we have rejected this bug.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: The diagrams I have shown in the screenshots are both class diagrams, so they are of the same type, hence is not related to the requirement you have pointed out.

To clarify my point, the problem I have raised in this bug report is that both these class diagrams are drawn in different styles which is confusing to the reader, hence a flaw in the documentation as documentations should strive to be clear and readable for others to understand your project. Documentation bugs do not have to affect the usage of the app to be accepted.


❓ Issue severity

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

  • I disagree

Reason for disagreement: [replace this with your explanation]


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants