Skip to content

Mentor workspace: Закрепление преподавателя за группами#670

Open
semrosin wants to merge 41 commits into
masterfrom
assign-lecturer-to-group-feature
Open

Mentor workspace: Закрепление преподавателя за группами#670
semrosin wants to merge 41 commits into
masterfrom
assign-lecturer-to-group-feature

Conversation

@semrosin
Copy link
Copy Markdown
Contributor

@semrosin semrosin commented May 6, 2026

Данный PR добавляет функциональность закрепления преподавателя за группами (добавления групп в фильтр):

  • Выбор группы в селекторе студентов. Группы подсвечиваются синим
Screenshot 2026-05-06 161411
  • После выбора группы ее студентов нельзя добавить по отдельности (студенты групп будут учитываться автоматически)
mentor_filter
  • Преподавателю, закрепленному за группой показываются только домашки этой группы (и общедоступные)
Screenshot 2026-05-06 162416
  • Выбор домашек только из тех, которые доступным выбранным группам
Screenshot 2026-05-08 120019
  • Решения тоже отображаются только от студентов, закрепленных за преподавателем
Screenshot 2026-05-06 162906
  • Группы и домашние задания теперь автоматически (при необходимости) добавляются в воркспейс преподавателя после создания

  • При получении MentorWorkspace поля, не выбранные явно преподавателем, возвращаются пустыми, а не содержащими всю информацию о курсе как раньше

Comment on lines +90 to +94
await props.onCreateNewGroup?.({
id: groupId,
name: formState.name.trim(),
studentsIds: formState.memberIds,
});
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

точно ли нам нужен этот метод?

Comment on lines +132 to +143
const courseData = await ApiSingleton.coursesApi.coursesGetAllCourseData(courseId)
const [courseData, mentorWorkspace] = await Promise.all([
ApiSingleton.coursesApi.coursesGetAllCourseData(courseId),
isNewHomework
? ApiSingleton.coursesApi.coursesGetMentorWorkspace(courseId, props.mentorId)
.catch(() => undefined)
: Promise.resolve(undefined)
]);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

давай перенесем логику работы с фильтрами на бекенд, фронтенд не должен знать про воркспейс

@semrosin semrosin force-pushed the assign-lecturer-to-group-feature branch from b4a04d3 to 0c9e148 Compare May 19, 2026 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants