search

nus-cs2113-AY1920S2 / pe-dev-response

0 stars 0 forks source link

Missing system architecture diagram #249

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.

image.png

DG seems incomplete as the system architecture diagram is missing and "TODO" is in place of it.

[original: nus-cs2113-AY1920S2/pe-interim#248]

JustinnT commented 4 years ago

Team's Response

Moving severity from Medium to VeryLow

Justification:

It is stated in the module website that Architecture diagram is optional. image.png

However, this bug is accepted as the TODO should have been removed. Because it is a cosmetic/typo error, it is changed to VeryLow as specified below.

severity.VeryLow : A flaw that is purely cosmetic and does not affect usage e.g., a typo/spacing/layout/color/font issues in the docs or the UI that doesn't affect usage. severity.Low : A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only.

Duplicate status (if any):

--