search

nus-cs2113-AY2223S2 / pe-dev-response

0 stars 0 forks source link

Constructor of command does calls execute() #1916

Open nus-pe-bot opened 1 year ago

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

Screen Shot 2023-04-14 at 5.56.17 PM.png

In this sequence diagram, the constructor of the "AddIncomeCommand" calls execute(), which in your code the reader does not think is the case.

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

Rayleigh47 commented 1 year ago

Team's Response

Fair point.

Duplicate status (if any):

--