Skip to content

feat: правка предпросмотра распределения экспертов - #815

Merged
TThanos3000 merged 1 commit into
devfrom
feat/dev-distribution-review-20261023
Oct 22, 2026
Merged

TThanos3000 merged 1 commit into
devfrom
feat/dev-distribution-review-20261023

Conversation

@TThanos3000

@TThanos3000 TThanos3000 commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Что изменено

POST expert-allocation принимает необязательные seed и assignment_overrides. Организатор может запросить другой сбалансированный вариант или поправить будущий состав проекта до подтверждения. Предпросмотр не меняет назначения, ответ API совместим с текущими клиентами.

Реализация / API

seed меняет порядок кандидатов только при одинаковой нагрузке. assignment_overrides заменяет предлагаемые добавления указанного проекта; сервер проверяет принадлежность программе, опубликованную сдачу, состав выбранного пула, кейс, собственную команду, дубли и свободные места. Недостаточный состав остается shortage и не применяется. Подписанный 15-минутный план, revision и атомарное применение сохранены.

UI

Отдельный frontend PR: полный состав вместо списка только добавлений, ручная правка и другой вариант. API выкладывается первым.

Permissions

Существующий manager/admin object scope без изменений. Модели и миграции не менялись.

Проверки

  • 54 PostgreSQL tests: case_distribution, workspace, workspace_concurrency.
  • Новые проверки: варианты seed, существующий + новый эксперт, preview без записи и apply, чужой кейс/пул/команда/проект, дубли, лимиты, shortage и stale revision.
  • Black и diff --check, review без blocking/major findings.

Acceptance criteria

  • Правка только будущих добавлений; существующие назначения и оценки сохраняются.

  • Пересчет и редактирование не пишут в базу до confirm.

  • Старые запросы работают без новых полей.

  • Both source GitHub CI workflows: 1323 full tests passed each; PostgreSQL extra 39 checks passed.

  • DEV deploy 39866129465: exact merge image, migrations/runtime access, HTTP smoke and Celery passed.

  • Frontend Added new vacancy fields #493 deployed; published-assets browser integration against isolated API passed 1440/390, including signed confirmation/read-back.

Не проверено

Реальные пользователи и письма исключены из QA. UI и опубликованная версия проверены с изолированным API; реальные DEV маршруты проверены анонимными GET.

Риски / migration

Миграции не нужны. DEV rollout API первым; production вне scope.

@TThanos3000
TThanos3000 merged commit 2e58389 into dev Oct 22, 2026
3 checks passed
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