search

nus-cs2113-AY2324S2 / pe-dev-response

0 stars 0 forks source link

UML diagram can be improved #1279

Open nus-pe-bot opened 2 months ago

nus-pe-bot commented 2 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.

Screenshot 2024-04-19 at 5.32.14 PM.png

View method command sequence diagram: the bottom part of the diagram, needs to be removed. Instances of the classes should be on top and not below.

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

nickczh commented 2 months ago

Team's Response

Dear Sir/Madam,

Thank you for pointing out your concern and we have classified this as not in scope as classes placed at the bottom is optional.

I hope this answers your query.

Thank you.

Regards, Eric

Duplicate status (if any):

--