search

Tetrerox / pe

0 stars 0 forks source link

Architecture is not updated to include the encryption component. #9

Open Tetrerox opened 2 years ago

Tetrerox commented 2 years ago

UG_bug3.1.jpg

UG_bug3.2.jpg

nus-pe-bot commented 2 years ago

Team's Response

Accepted. However, we argue that the severity of this is very low because it will not affect the developer very much.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: I think that the severity should remain at least low. The architecture diagram is supposed to give readers a high level overview of the implementation of the software but missing out on including one component is misleading to the readers and can hinder the mental mindmap of the implementation that the readers have in their minds. It is not a simple typo that is severity.VeryLow.