search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Mislabelled sequence diagram (p11) #1833

Open nus-pe-bot opened 3 months ago

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

Description

Screenshot 2024-04-19 at 17.29.38.png

Predicate shown in the sequence diagram is under Logic, however, in the codebase, the Predicates are found under Model.

Screenshot 2024-04-19 at 17.30.42.png

Fix

Move the predicate class under the model section.

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

zhekaiii commented 3 months ago

Team's Response

In the CS2103T syllabus, it is not specified that boxes must refer to the package of the classes in that box.

Duplicate status (if any):

--