search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

DG Model.java Diagram Error #5105

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

In this diagram, the multiplicities imply that a timetable can have between 5 and 7 days, but the program only allows Mondays through Fridays slots to be saved, no weekends.

Same for period, the hours can only be between 0 and 23 (12am and 11pm), hour 0000 and hour 2400 are the same!

image.png

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

tanjiajiajun commented 5 months ago

Team's Response

The implementation for accepting 7 days is already implemented, just that the option to enable it is not available to the users. Hence, in developer's guide, it is shown as 5 or 7 days, given the fact that it is already implemented, the developer should be made known to it. On top of that, it is specified in the User Guide that it is a planned enhancement.

Duplicate status (if any):

Duplicate of #2681