search

nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

[DG] Erroneous sequence diagram in DG #665

Open nus-se-script opened 1 week ago

nus-se-script commented 1 week ago

image.png

The arrow inside the "alt" box, for "throw CommandException"... starts from an ambiguous place. It is unclear whether it is Model or Person that is throwing the Exception.

Another suggestion is that, considering the complexity of the diagram, minimal notation of the sequence diagram may not be advisable as there is some ambiguity involved.

[original: nus-cs2103-AY2425S1/pe-interim#944] [original labels: severity.Medium type.DocumentationBug]

QuekBingHeng commented 1 week ago

Team's Response

Duplicate of #1572.

Duplicate status (if any):

Duplicate of #1572