nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Activity diagram shows only one lesson being added to the timetable #418

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year 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.


Activity diagram shows only one lesson being added to the timetable although header states that this is for adding possibly multiple lessons. Furthermore the end of the activity diagram shows the overall timetable string being returned, again indicating that all lessons should be added within this activity diagram. To add multiple activities, a looping structure is expected.

The [is X written one row below] early termination is also confusing because it means that the remaining logic like "add clash modules at bottom" and "return output string" are not executed.

image.png


[original: nus-cs2113-AY2223S1/pe-interim#435] [original labels: severity.Low type.DocumentationBug]

HT-T commented 1 year ago

Team's Response

No details provided by team.

Duplicate status (if any):

--