lckjosh / pe

0 stars 0 forks source link

Circle shown in UML diagram #15

Open lckjosh opened 11 months ago

lckjosh commented 11 months ago

Screenshot 2023-11-17 at 5.19.20 PM.png

This circle should be hidden as it does not comply with the notation taught in this course.

nus-se-bot commented 11 months ago

Team's Response

duplicate of 499

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Non-standard notation UML (use of symbols in class header) [DG page 6]

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.


image.png

Should not be any C symbols following the standard notation, such as the below.

image.png


[original: nus-cs2113-AY2324S1/pe-interim#684] [original labels: severity.Low type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you for your response.
Will take note of this, and make the necessary improvements in the future.

The "C" symbol refers to class.
Should have had more clear label

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.Rejected`] - [x] I disagree **Reason for disagreement:** I would consider this a typo and thus a bug. It should not be rejected. However, I agree that my original severity rating of "medium" is too high. I think it should be low.
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]