Add and fix moderator scheduling checks #3678
Conversation
There was a problem hiding this comment.
I think this looks good so far. There is an edge case of when someone is moderating and teaching at the same time. The checks show that there is a teacher teaching when they aren't available, but the moderator check does not show an issue. I actually think neither of these should show an issue and that it should show the issue under double booked teachers (it doesnt). Perhaps we also want to add a double booked moderator check, or just include them in the double booked teacher check (my preference, i think).
esp/templates/program/modules/ajaxschedulingmodule/ajax_scheduling.html
Outdated
Show resolved
Hide resolved
|
Thanks for the great comments, @kkbrum! I believe I've addressed all of them. Teacher and moderator availability checks now skip the other availabilities, while the double-booked check now covers both. |
* Docs for #3670, #3672, #3671, and #3676 * Docs for #3677, #3673, #3681, and #3683 * Docs for #3682, #3678, #3684, and 74b7e0f * Docs for 74b7e0f, #3685, #3687, and #3686 * Docs for #3688, #3694, #3695, #3692, and #3691 * Docs for #3642, #3696, #3697, and #3700 * change date and add customized buttons for bigpicture * add theme help text --------- Co-authored-by: Katherine <[email protected]>
This adds a moderator availability scheduling check to the ajax scheduler and the scheduling checks page. It also fixes the moderator title for all checks in both of these places.
I also adjusted the coloration of the cells for the teacher/moderator availability checks to be based on the proportion of teachers that are unavailable (1/1 unavailable is the same color as 5/5 unavailable, which are both much darker than 1/5 unavailable).
Note, I also fixed the tab spacing in one of the files (hence why the diff looks like I changed a lot).
Fixes #3665.