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.

Sign up for GitHub

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

Jump to bottom

EntryList UML diagram has incorrect UML notation #14

Open
Arif-Khalid opened this issue Apr 14, 2023 · 1 comment
Open

EntryList UML diagram has incorrect UML notation #14

Arif-Khalid opened this issue Apr 14, 2023 · 1 comment
Labels
severity.Low status.Done type.DocumentationBug

Comments

@Arif-Khalid
Copy link
Owner

Arif-Khalid commented Apr 14, 2023 •
edited
Loading

The dotted arrows are meant to display dependencies, the arrow to entry should be solid to show reference.
image.png
@nus-pe-script
Copy link

nus-pe-script commented Apr 18, 2023 •
edited by Arif-Khalid
Loading

Team's Response

They are indeed supposed to be dotted lines as the EntryList class does not contain any attributes of Entry or Category, but they make use of them in the methods.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: Dotted arrows should then not be accompanied by multiplicity, dependencies with multiplicity is clearly an error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
severity.Low status.Done type.DocumentationBug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nus-pe-script @Arif-Khalid