search

matthewc97 / pe

0 stars 0 forks source link

Lack of class diagram #8

Open matthewc97 opened 4 years ago

matthewc97 commented 4 years ago

image.png

Would be nice to include class diagram to show the relationship between the two classes

nus-pe-bot commented 4 years ago

Team's Response

This bug is rejected because of how the classes are implemented for Version 2.1. For now, there isn't a direct correlation between Patient and appointment classes other than the PatientID which is called when AddAppointmentCommand is called (checkPatientIdUsed method). Drawing one would only be causing repetition in the lengthy DG. Also, reduced it to low severity because, more information would be given in AddAppointmentCommand.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: [replace this with your explanation]