search

zlimez / pe

0 stars 0 forks source link

UML mislabel #9

Open zlimez opened 1 year ago

zlimez commented 1 year ago

From what I gathered in the application, there is no entity called "Person" so likely the deletePerson method does not exist. Perhaps the diagram can be updated.

Screenshot 2022-11-11 at 5.29.15 PM.png

Screenshot 2022-11-11 at 5.33.54 PM.png

soc-pe-bot commented 1 year ago

Team's Response

As seen in https://github.com/AY2223S1-CS2103T-F12-3/tp/blob/master/src/main/java/seedu/address/model/Model.java#L73

image.png

So the model is technically correct. However, we do recognize that this should be changed, and so we are marking this as Not In Scope rather than Rejected.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]