search

nus-cs2103-AY1920S2 / pe-dev-response

0 stars 0 forks source link

wrong arrow in figure 6 class diagram #2463

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

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.

the inheritance arrow has no start class. should be the interviewPanel inherits UiPart.

image.png

[original: nus-cs2103-AY1920S2/pe-interim#2445]

bscrow commented 4 years ago

Team's Response

Accepted, as this is due to a mix up between the notation requirement of the architecture diagram and the class diagram. This is intended to show that all classes in the diagram inherits from UiPart, because connecting UiPart to all classes in the diagram will get penalised for messiness (InterviewPanel is composed ogf 8 UI components afterall)

Based on the bug severity labels (reproduced below), this bug should be classified as Severity.Low as it is unlikely to affect normal operations of the product.

image.png

Duplicate status (if any):

--