Skip to content

DEV: убрать прогресс критериев из назначений экспертов - #746

Merged
TThanos3000 merged 1 commit into
devfrom
fix/dev-assignment-criteria-progress
Sep 20, 2026
Merged

TThanos3000 merged 1 commit into
devfrom
fix/dev-assignment-criteria-progress

Conversation

@TThanos3000

Copy link
Copy Markdown
Collaborator

Из публичных данных назначения удалены criteria_total и criteria_scored: счётчик заполненных критериев в списке дублировал статус назначения. Изменение касается списка назначений и унаследованного контракта detail /scores/.

Проверяемая версия

  • Base (dev): dd3d48b121a1fd0668857c9b8766bb588be2360b.
  • Head: e4992d18d772849f26e419799be11c35f807c18b.
  • Ветка: fix/dev-assignment-criteria-progressdev.

Что изменено

  • ProgramAssignmentSerializer больше не публикует два агрегированных поля.
  • build_assignment() исключает их из возвращаемого словаря; внутренние annotations и расчёт статуса сохранены.
  • ProgramAssignmentScoresSerializer по-прежнему наследует назначение и добавляет массив scores со всеми критериями, значениями и is_scored. Endpoint и права доступа сохранены.
  • Обновлены строгие тесты контракта и документация API.

Completion rule, статусы not_ready/pending/in_progress/completed, scopes, waiting/SLA, delayed experts и overview counters не менялись. Показатель завершённых назначений проекта сохраняется. React, PROD, модели, миграции, зависимости и workflows не затронуты.

Проверки

  • Targeted PostgreSQL: python manage.py test partner_programs.tests.test_assignment_analytics_api partner_programs.tests.test_manager_analytics_api partner_programs.tests.test_attention_analytics_api --verbosity 1 --keepdb --noinput111/111.
  • Full PostgreSQL: python manage.py test --verbosity 1 --keepdb --noinput867/867, exit 0.
  • python manage.py check — PASS.
  • Black и flake8 изменённых Python-файлов — PASS.
  • git diff --check — PASS.

Проверены отсутствие полей в service/list/detail, нулевые и частичные оценки, все критерии, несданный проект, три scope, прежние SLA/counters и доступ. SQL-бюджеты остались 3/14/5 для list/overview/detail и не растут с числом назначений.

Проверки выполнялись в отдельной локальной PostgreSQL БД. Для совместного будущего DEV-выпуска подготовлен отдельный Angular Draft PR, удаляющий колонку; старый Angular ожидает удаляемые поля, поэтому backend не следует выпускать отдельно до согласования порядка. Merge и deploy не выполнялись.

@TThanos3000
TThanos3000 marked this pull request as ready for review September 20, 2026 09:53
@TThanos3000
TThanos3000 merged commit af8be95 into dev Sep 20, 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