Skip to content

Add submitted evaluation amendments - #673

Merged
TThanos3000 merged 1 commit into
masterfrom
feature/dev-073-evaluation-amendment-api
Jul 29, 2026
Merged

Add submitted evaluation amendments#673
TThanos3000 merged 1 commit into
masterfrom
feature/dev-073-evaluation-amendment-api

Conversation

@TThanos3000

Copy link
Copy Markdown
Collaborator

Что изменено

  • Добавлен отдельный сценарий изменения уже отправленной экспертной оценки.
  • Добавлен endpoint PATCH /evaluations/{evaluation_id}/amend/.
  • Добавлен endpoint истории GET /evaluations/{evaluation_id}/amendments/.
  • Evaluation остаётся в статусе submitted, исходный submitted_at сохраняется.
  • Добавлен amended_at; при фактическом изменении total_score сбрасывается в null до отдельного расчёта.
  • Scores заменяются полным набором в одной транзакции и повторно валидируются по критериям программы.
  • Повторный идентичный запрос не создаёт лишнюю запись истории.
  • Добавлена модель EvaluationAmendment со снимками значений до и после изменения.
  • История доступна владельцу оценки, менеджеру программы и staff/superuser; посторонним возвращается 404.
  • Добавлены отдельный throttle, read-only admin, миграция и документация.

Проверки

  • 63 regression-теста экспертного контура пройдены, 1 ожидаемо пропущен на SQLite из-за отсутствия select_for_update.
  • 11 тестов DEV-073 пройдены.
  • manage.py check --tag models — успешно.
  • makemigrations --check --dry-run — изменений нет.
  • Black, flake8 и git diff --check — успешно.

Развёртывание

После merge применить только на React-dev backend /root/api-react-dev:

  • обновить код;
  • пересобрать и пересоздать React-dev web/celery контейнеры;
  • выполнить миграцию 0024_evaluation_amended_at_evaluationamendment;
  • проверить amendment API на PostgreSQL.

Production и старый dev backend /root/api не изменять.

Roadmap-IDs: DEV-073

@TThanos3000
TThanos3000 merged commit 5f81ccf into master Jul 29, 2026
4 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.

1 participant