search

JoelHo / pe

0 stars 0 forks source link

Architecture->Model has not been updated from AB-3 #21

Open JoelHo opened 3 years ago

JoelHo commented 3 years ago

The Model section under architecture only shows Person, however, the product has Student, Tutor and Session, none of which are indicated on the diagram

image.png

nus-pe-bot commented 3 years ago

Team's Response

These are the actual class names in the backend code.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: While those may be the right classnames, it is missing vital details on the tutors/student and session. Surely a session is not a Person. It also makes no mention of whether a tutor/student is a Person, and how that is modeled (as a field/subclass/?).