search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Contradiction with regards to what deletePerson() takes in #4098

Open nus-se-script opened 5 months ago

nus-se-script commented 5 months 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.

image.png

image.png

Difference in sequence diagram, in terms of what deletePerson() method takes in. In one diagram, deletePerson takes in a string "Alex Yeoh" while in another it take in a person object p.

[original: nus-cs2103-AY2324S2/pe-interim#5151] [original labels: severity.Low type.DocumentationBug]

yapxuanxuan commented 5 months ago

Team's Response

Good to fix in future versions.

Duplicate status (if any):

--