search

nus-cs2113-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Missing return arrow in ComposeCommand #100

Open nus-pe-bot opened 3 years ago

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

Although the methods may not return a value but it should still show a return arrow for the methods: addToDraft(), updateAllTypeEmails(), printEmailDrafted() This show the control is return to back to the ComposeCommand and also outside of the ComposeCommand.

image.png

image.png

[original: nus-cs2113-AY2021S2/pe-interim#100] [original labels: severity.VeryLow type.DocumentationBug]

jalvinchan commented 3 years ago

Team's Response

Not drawing return arrow is valid in sequence diagram and is meant to simplify the diagram.

Duplicate status (if any):

--