Skip to content

fix: подтверждение исключения из команды проекта-заявки - #473

Merged
TThanos3000 merged 2 commits into
devfrom
fix/dev-application-team-removal-copy-20261021
Oct 21, 2026
Merged

TThanos3000 merged 2 commits into
devfrom
fix/dev-application-team-removal-copy-20261021

Conversation

@TThanos3000

@TThanos3000 TThanos3000 commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Что изменено

Окно удаления участников теперь показывает конкретные команды проектов-заявок и имена исключаемых участников. Оно объясняет, что команда исходного проекта сохраняется, а оценки уже сданной заявки не меняются. Подтверждение отправляет confirmation_version:2.

Зачем

Заявка в программе — отдельная копия проекта. Участника нужно исключать из ее команды, сохраняя исходный проект и аккаунт. Прежний текст обещал сохранить все команды и больше не соответствует выбранному поведению.

Реализация

Расширены существующие domain preview model, HTTP adapter и centered confirmation modal. Новый визуальный компонент не создается; тонкие кнопки и плавающие массовые действия сохраняются.

API

Зависит от PROCOLLAB-github/api#810. API-first deployment: remove-preview возвращает confirmation_version:2 и team_changes, remove требует confirmation_version:2. Backend отклоняет старый интерфейс до его обновления, чтобы не расширять эффект удаления без нового подтверждения.

UI

Показаны название заявки, участники, отзыв приглашений и сохранение оценок. Сохранено предупреждение при удалении руководителей. Проверены1440/390/360px, тонкие кнопки, centered/scrollable modal, отсутствие горизонтального overflow и сдвига строк при выборе.

Permissions

Server-side permissions и удаление выполняются API. Клиент показывает подписанный preview и отправляет подтверждение. Глобальный ProjectID и ID заявки не смешиваются.

Проверки

  • Checked by source: contract, layer boundaries, rollout order и requirement audit.
  • Checked by tests:36 relevant management DOM/HTTP adapter tests passed.
  • Checked by build: npm run build:pr passed.
  • Changed-file ESLint/Prettier и git diff check passed.
  • Checked in browser: local Angular build + isolated Django/PostgreSQL + locmem mail;6 draft/submitted scenarios at1440/390/360px passed. Preview/cancel/delete, source preservation, private application/chat403, source200, remaining-team submit, retained score7/assignment, leader warning;0 runtime/server errors.
  • Full frontend440 suites2327 tests, format/lint/build/rsync passed; published1440/390/360 all6 workflows and14 actual anonymous DEV routes passed.

Acceptance criteria

  • AC1–AC7 server behavior covered by API PR810 tests and local browser integration.
  • AC8 explicit confirmation, thin controls and stable selection on desktop/mobile.

Не проверено

Настоящие DEV users и SMTP намеренно не изменяются. Business writes в browser QA идут только в локальный API; actual DEV защищенные маршруты проверяются анонимными GET.

Риски / migration

Без миграций. Merge/deploy после успешного API PR810 deployment. Production не затрагивается.

DEV deployments completed: API PR810 →1c424ff7 (run39587159832), frontend PR473 →cc95cda8 (run39587847132). Merge trees match tested sources. Production, real users and SMTP unchanged.

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