search

StanleyNeoh / pe

0 stars 0 forks source link

Missing argument in sequence diagram in the developer guide #15

Open StanleyNeoh opened 1 year ago

StanleyNeoh commented 1 year ago

image.png

Based on the above code here from LogicManager.java. the Command#execute(Model) method takes in a Model instance.

However, the following diagram shows no Model argument being shown here.

image.png

More specifically this particular call.

image.png

This can be confusing to the developer reading the developer guide.

nus-pe-script commented 1 year ago

Your response not required for this bug as the team has accepted the bug as it is.