search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

File Management Sequence Diagram #306

Open nus-pe-bot opened 5 months ago

nus-pe-bot commented 5 months 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

Is EconoCraftLogic a object or is the diagram showing a static method? It should be indicated as such either

static: image.png

object: With a : (colon)

Eg. image.png

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

StargazerX0 commented 5 months ago

Team's Response

Thanks for the report and suggestion! EconoCraftLogic is an object and there is a typo with the colon during generation of UML. Since this is a typo which affects the actual content of diagram slightly, we would like to adjust the issue's severity to VeryLow.

Duplicate status (if any):

--