Skip to content

fix: понятное распределение экспертов и статусы приглашений - #446

Merged
TThanos3000 merged 1 commit into
devfrom
fix/dev-experts-clarity-20261018
Oct 18, 2026
Merged

TThanos3000 merged 1 commit into
devfrom
fix/dev-experts-clarity-20261018

Conversation

@TThanos3000

Copy link
Copy Markdown
Contributor

Что изменено

Раздел «Эксперты» объясняет, сколько оценок нужно каждому проекту и для скольких проектов ещё нужно подобрать экспертов. Аналитика показывает актуальное обязательное число оценок. Отправка письма обновляется автоматически и отделена от принятия приглашения.

Зачем

Убрать непонятные «не хватает 1 назначений», расхождение N с аналитикой и застывшее «Отправляется». В автоматическом распределении теперь явны выбор экспертов, предпросмотр и подтверждение.

Реализация

Copy/status states в ManagementExperts; ограниченный polling с отменой, timeout и сохранением фокуса/поиска; analytics model/tooltips; совместимые optional поля. Фиктивный select единственного варианта убран, в выборе только принявшие приглашение эксперты.

API

Использует совместимые поля API PR PROCOLLAB-github/api#789. Сначала backend. Payloads, авторизация, revision и правила распределения не менялись.

UI

Тот же утверждённый shell, Mont, DS controls и модалки по центру. Проверены desktop 1440 и mobile 390/360; инструкция сокращена, кнопки не сжимаются.

Permissions

Не менялись; ограничения программы/команды/сохранённых оценок остаются на API.

Проверки

  • Checked by tests: 64 targeted tests экспертов и аналитики (в том числе старый режим).
  • Checked by build: build:social:dev, success.
  • Prettier, ESLint и Stylelint затронутых файлов; полный ESLint 0 errors / 6 baseline warnings; diff --check.
  • Checked in browser: 8 сценариев с реальным изолированным Django/PostgreSQL; queued→sent через настоящий task с locmem mail, mixed-case имя, N 3→1, аналитика, preview/apply, адаптив.
  • Checked by source: requirement audit, backward compatibility, cancellation и scope diff.

Acceptance criteria

  • Число оценок, нехватка экспертов и завершённость проектов различимы.
  • Аналитика показывает актуальное N.
  • Распределение объяснено по шагам, запись происходит только при подтверждении.
  • Письмо имеет отдельный статус и обновляется без перезагрузки/потери ввода; длительное ожидание имеет выход.

Не проверено

Внешняя доставка во входящие не проверялась. Повторная проверка конкретных данных dev выполняется после rollout; массовая правка старых аккаунтов не входит в UI fix.

Риски / migration

Миграций нет. Optional response fields совместимы с поэтапной выкладкой; backend PR #789 первым. Polling работает только для очереди письма, ограничен по времени и не сбрасывает открытые формы.

@TThanos3000
TThanos3000 merged commit 433d099 into dev Oct 18, 2026
@TThanos3000
TThanos3000 deleted the fix/dev-experts-clarity-20261018 branch October 18, 2026 11:21
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