GradeBookListView для преподавателей должно показывать все доступные курсы
Closed this issue · 0 comments
pacahon commented
Сейчас логика такая, что для teaching.views.GradeBookListView в выборку не попадают все летние семестры. Хотя по-хорошему это ограничение там не нужно и можно код сделать более универсальным. Мотивация понятна - если преподаватель по каким-то причинам ведёт курс летом, то и ведомость он должен видеть.
Причина, почему нет летних курсов - teaching.views.GradeBookListView наследуется от GradeBookListBaseView, которая используется и для страницы ведомостей кураторов. Но на странице кураторов не нужно показывать летние семестры, поэтому в выборку они не попадают.
- Разделить логику GradeBookListBaseView для страницы ведомостей преподавателей и кураторов
- Убрать mock из теста learning.teaching.tests.test_views.test_gradebook_list