nus-cs2103-AY2425S1 / pe-dev-response

0 stars 0 forks source link

Makeup lesson is automatically scheduled for all students in the group #3479

Open nus-se-script opened 6 days ago

nus-se-script commented 6 days ago

Screenshot 2024-11-15 at 5.02.00 PM.png

In the event 1/3 students did not attend the lesson, it is unlikely that the whole group will be given makeup lesson. Makeup lessons should thus be more flexible and allow scheduling of individual students rather than the group. Furthermore, if a teacher has many groups, cancelling might not be an option since it is difficult to keep rescheduling a group of students and to find a common time outside of regular class timings to give makeup to the whole group.


[original: nus-cs2103-AY2425S1/pe-interim#2668] [original labels: severity.Medium type.FeatureFlaw]

IzN432 commented 5 days ago

Team's Response

This is an intentional design choice. We considered the alternative of allowing single student makeup lessons, but there are two issues with this

  1. Scheduling a makeup lesson for an entire group would take much longer
  2. Should we allow the makeup lessons of two students in the same group to clash

Ultimately, we concluded that it is more likely that a makeup lesson would be scheduled for the entire group. There is also currently a workaround as well, to simply create makeup lessons for the entire group instead, although it will show up for all students in the group.

In the end, this alternate implementation would take more effort to implement and we have chosen to focus on other more important features.

Duplicate status (if any):

--