Shruthiiii03 / pe

0 stars 0 forks source link

Multiplicities not stated for student attributes #17

Open Shruthiiii03 opened 2 weeks ago

Shruthiiii03 commented 2 weeks ago

Screenshot 2024-11-15 at 5.18.12 PM.png

Multiplicities are not specified for student attributes like tutorial group and student number. It is ambiguous whether a student can have multiple tutorial groups for example or only 1 tutorial group. Especially in the case where a student can only have one student number, the multiplicity should be depicted as one. It is also hard to tell whether certain attributes like the assignment list are optional as the multiplicity is not stated.

nus-pe-script commented 1 week ago

Team's Response

This is an oversight on our part, thanks for the feedback! However, we do not believe this would heavily impede a developer from developing the application.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: While I agree that this issue may not significantly hinder a developer from building the application, it does have the potential to cause minor confusion, particularly when dealing with optional fields like ObservableList and AttendanceRecordList. As such, I believe the severity level should be adjusted to severity.Low rather than dismissed as merely a cosmetic issue. This is because the confusion arising from unclear implementation details could still impact developers' understanding and usage of these attributes, albeit to a limited extent.