Add gradebook ref to courses; Add Useful to teaching tab; Color panel headings of graduated student profiles; Add html parse to honor code #801
Conversation
| все семестры</a> | ||
| {% if user.is_curator or user in course.teachers.all()%} | ||
| <br> | ||
| <a href="{{request.build_absolute_uri(url('staff:gradebook', **course.url_kwargs))}}"> |
There was a problem hiding this comment.
А почему тут важно именно absolute_uri?
There was a problem hiding this comment.
Потому что ссылка ведет из курса на ведомость. То есть совсем в другую ветку. Можно конечно обойтись без этого, но указать абсолютный адрес для ссылки мне кажется самым простым и одновременно лаконичным решением.
|
|
||
| class TeachingUsefulListView(PermissionRequiredMixin, generic.ListView): | ||
| context_object_name = "faq" | ||
| template_name = "learning/study/useful.html" |
There was a problem hiding this comment.
Не вижу этого template в PR
There was a problem hiding this comment.
Да, я переиспользовал template, который используется для "Полезного" для студентов. Запрос был на то же самое, только с другим контентом. Поэтому изменения в одном шаблоне должно логично касаться и второго.
| @@ -53,6 +53,7 @@ | |||
| path('', RedirectView.as_view(pattern_name='teaching:assignments_check_queue', permanent=False), name='base'), | |||
| path('timetable/', TeacherTimetable.as_view(), name='timetable'), | |||
| path('calendar/', CalendarPersonalView.as_view(), name='calendar'), | |||
| path('useful/', TeachingUsefulListView.as_view(), name='teaching_useful'), | |||
There was a problem hiding this comment.
Все другие пути содержат имя вью более точно. Переименуем?
There was a problem hiding this comment.
По моему путь выдержан в одном тоне с остальными. Также как TeacherTimetable превратился в timetable/ и CalendarPersonalView в calendar/, новый путь оставил только одно слово, отображающее суть. Также и "Полезное" для студентов при вьюшке UsefulListView использует путь useful/.
| @@ -19,7 +19,13 @@ <h2 class="course-main-title"> | |||
| <a href="{{ url('admin:courses_course_change', object_id=course.pk) }}" target="_blank"><i | |||
| class="fa fa-pencil-square-o"></i></a>{% endif %}<br> | |||
| <small>{{ course.main_branch.name }} / {{ course.semester }}, <a href="{{ course.meta_course.get_absolute_url() }}">посмотреть | |||
There was a problem hiding this comment.
Мне не нравится, что тег <small> не просто вокруг одного блока, а сразу вокруг двух.
Я думаю, что в этом случае лучше каждый тег <a> обрамить отдельным <small>.
Checklist:
printstatements, unrelated template context variables, commented code, etc)