search

Ziyang-98 / pe

0 stars 0 forks source link

Incorrect notations in class diagram under Statistics feature in DG #13

Open Ziyang-98 opened 4 years ago

Ziyang-98 commented 4 years ago

I think it may be incorrect to include . If it is necessary, it should be represented as a association in the class diagram.

Screenshot 2020-11-13 at 5.40.44 PM.png

nus-se-bot commented 4 years ago

Team's Response

We are unable to understand what you mean by "I think it may be incorrect to include .".

Items for the Tester to Verify

:question: Issue response

Team chose [response.IssueUnclear]

Reason for disagreement: I apologize if the issue is unclear. What I meant was "UsageList" should use associations to indicate the (T extends Entry) instead of including it in the diagram as it may be an incorrect notation. However, I did implied that together with the screenshot I posted. Additionally, I did include "(T extends Entry)" in the PE description with arrow brackets but it was magically removed from the description. I suspect that it might be a bug in Catcher when dealing with arrow brackets. I highlighted it in forums as well.

(Update from the forums) Prof Damith and a friend highlighted that it is a Github Behaviour from Github Markdown and thus, the arrow bracket and the content is regarded as an invalid tag. Therefore, it is not shown in the description.