search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Sequence diagram AddCommand #4079

Open nus-se-script opened 2 months ago

nus-se-script 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.

image.png

hs:HashSet is not found in the code. In the code, it is named assignmentSet image.png

May cause confusion to future developers

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

thewongdylan commented 2 months ago

Team's Response

Sequence diagrams are meant to show high level implementations, hence there is no need to show exact variable name as per the actual code.

Duplicate status (if any):

--