nus-cs2113-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Able to add two modules with classes at the same time #697

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Since the program adds the first of every instance of the tutorial, lab, and lecture to the timetable, it does not account for any overlap in meeting times for them when entering the time values. In this scenario, the 1st tutorial for cs2113 and the lab/tutorial slot for ma2213 have the same first instance of tutorial as the first instance of the lab for cs3103, from 1200-1300 and from 1300-1400, and there is no indication of that being a problem when initially adding to the timetable.

Screenshot 2022-11-11 at 4.56.04 PM.png


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

owenl131 commented 1 year ago

Team's Response

When viewing their timetable, since it is shown in a graphical format, the user can clearly see the clash, hence there is no need to show an error message to frighten the user.

A timeslot clash is not considered a problem, it is simply an expected intermediate step within the timetable planning process.

Duplicate status (if any):

--