EthanLoww / pe

0 stars 0 forks source link

Delete Command: Sequence Diagram is wrong #5

Open EthanLoww opened 2 weeks ago

EthanLoww commented 2 weeks ago

The deletePerson should take in a Person object instead of an index. However, the sequence diagram below shows an index "1" being passed into the deletePerson method.

image.png

This contradicts the below image, where a Person p is passed into the deletePerson method.

image.png

nus-pe-script commented 1 week ago

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