Диссертационный совет: сортировка по ФИО и количеству публикаций #58

Merged
admin merged 1 commits from feature/council-sorting into main 2026-10-08 07:26:19 +00:00
Collaborator

На публичной странице /admin/council добавлены видимые поля «Сортировать по» и «Направление». Доступны ФИО, публикации CSV по критериям, публикации в БД и общее количество без дублей; каждый вариант работает по возрастанию и убыванию. Кнопка «Применить» одновременно применяет поиск и сортировку.

Сортировка выполняется для всего списка до пагинации, сохраняется при переходе между страницами, поиске и его сбросе. По умолчанию сохраняется прежний порядок: больше подходящих публикаций CSV, затем больше работ всего. При равных числах порядок детерминирован по ФИО. Неизвестные параметры заменяются безопасными значениями по умолчанию. Использованы существующие стили и нативные select без новых зависимостей. README и frontend/backend/package версии обновлены до 0.12.0.

Проверки: pytest — 95 passed, 10 skipped (PostgreSQL integration tests без тестовой БД); Playwright browser_council.py — passed. Проверены все четыре поля в обоих направлениях, равные счётчики, объединение дублей, неверные параметры, сортировка до пагинации, сохранение выбора при поиске/сбросе/пагинации и отсутствие переполнения на ширине 320–1440 px. node --check admin.js, sh -n обоих backup-скриптов и git diff --check — passed. Детектор Impeccable не сообщил находок. Скриншоты на синтетических данных: sorting-desktop.png и sorting-mobile.png в корне локального проекта; также .impeccable/review/.

Ожидается независимое ревью в отдельной сессии. Production не изменён. Changelog — отдельный шаг после готовности PR согласно установленному порядку.

На публичной странице /admin/council добавлены видимые поля «Сортировать по» и «Направление». Доступны ФИО, публикации CSV по критериям, публикации в БД и общее количество без дублей; каждый вариант работает по возрастанию и убыванию. Кнопка «Применить» одновременно применяет поиск и сортировку. Сортировка выполняется для всего списка до пагинации, сохраняется при переходе между страницами, поиске и его сбросе. По умолчанию сохраняется прежний порядок: больше подходящих публикаций CSV, затем больше работ всего. При равных числах порядок детерминирован по ФИО. Неизвестные параметры заменяются безопасными значениями по умолчанию. Использованы существующие стили и нативные select без новых зависимостей. README и frontend/backend/package версии обновлены до 0.12.0. Проверки: pytest — 95 passed, 10 skipped (PostgreSQL integration tests без тестовой БД); Playwright browser_council.py — passed. Проверены все четыре поля в обоих направлениях, равные счётчики, объединение дублей, неверные параметры, сортировка до пагинации, сохранение выбора при поиске/сбросе/пагинации и отсутствие переполнения на ширине 320–1440 px. node --check admin.js, sh -n обоих backup-скриптов и git diff --check — passed. Детектор Impeccable не сообщил находок. Скриншоты на синтетических данных: sorting-desktop.png и sorting-mobile.png в корне локального проекта; также .impeccable/review/. Ожидается независимое ревью в отдельной сессии. Production не изменён. Changelog — отдельный шаг после готовности PR согласно установленному порядку.
codex added 1 commit 2026-10-07 13:50:56 +00:00
feat: add council employee sorting by name and publication counts
CI / test (pull_request) Successful in 9m15s
CI / Fingerprint diagnostics (production deployment temporarily disabled) (pull_request) Has been skipped
32e26ccdd2
agent approved these changes 2026-10-08 07:26:08 +00:00
agent left a comment
Collaborator

Независимое ревью коммита 32e26ccdd2 завершено. Замечаний, требующих исправления, не обнаружено.

До проверки подтверждено совпадение origin с репозиторием PR. Через GET /user установлена независимая Gitea-идентичность: reviewer agent (id 7), автор PR и коммита codex (id 6).

Изучены все 9 изменённых файлов и связанный поток route → council_candidates → template, существующие стили и footer. Проверены допустимые поля и направления, безопасные значения для неизвестных параметров, сортировка полного списка до пагинации, вторичные ключи, сохранение выбора при поиске/сбросе/пагинации, экранирование и доступные подписи select. Версии согласованы на 0.12.0. Тесты покрывают четыре поля в обоих направлениях, равные счётчики, дедупликацию и пагинацию; browser-сценарий также проверен чтением.

Проверка git diff --check прошла. Gitea сообщает success для CI / test на этом SHA; workflow включает pytest и синтаксические проверки JS и backup-скриптов. Production diagnostics пропущены по условию workflow. Локальные тесты и браузер не запускались: сессия reviewer ограничена fetch/read/diff/comment/approval. Результаты Playwright из описания PR независимо запуском не подтверждались.

Код не изменялся, push и merge не выполнялись.

Независимое ревью коммита 32e26ccdd2b691ca6f04f49bbde0554df2f287af завершено. Замечаний, требующих исправления, не обнаружено. До проверки подтверждено совпадение origin с репозиторием PR. Через GET /user установлена независимая Gitea-идентичность: reviewer agent (id 7), автор PR и коммита codex (id 6). Изучены все 9 изменённых файлов и связанный поток route → council_candidates → template, существующие стили и footer. Проверены допустимые поля и направления, безопасные значения для неизвестных параметров, сортировка полного списка до пагинации, вторичные ключи, сохранение выбора при поиске/сбросе/пагинации, экранирование и доступные подписи select. Версии согласованы на 0.12.0. Тесты покрывают четыре поля в обоих направлениях, равные счётчики, дедупликацию и пагинацию; browser-сценарий также проверен чтением. Проверка git diff --check прошла. Gitea сообщает success для CI / test на этом SHA; workflow включает pytest и синтаксические проверки JS и backup-скриптов. Production diagnostics пропущены по условию workflow. Локальные тесты и браузер не запускались: сессия reviewer ограничена fetch/read/diff/comment/approval. Результаты Playwright из описания PR независимо запуском не подтверждались. Код не изменялся, push и merge не выполнялись.
admin merged commit 6bd1f31f3a into main 2026-10-08 07:26:19 +00:00
Collaborator

Повторное независимое ревью PR #58, head 32e26ccdd2: замечаний, требующих исправления, не обнаружено. PR уже слит; существующее APPROVED для этого SHA сохраняется.

Подтверждены совпадение origin с admin/miem_workers и независимость Gitea-аккаунта agent (id 7) от автора PR codex (id 6). Изучены все 9 изменённых файлов и связанный поток route → council_candidates → template, стили и footer. Проверены разрешённые параметры сортировки, значения по умолчанию, стабильная сортировка по вторичным ключам, применение до пагинации и сохранение выбора при поиске, сбросе и переходах. Версии согласованы на 0.12.0. Изучены unit- и browser-сценарии.

git diff --check прошёл. Через API подтверждён success для CI / test на проверенном SHA; workflow включает pytest и синтаксические проверки. Локальные тесты и браузер не запускались в рамках ограничений reviewer-сессии. Fetch недоступен из-за read-only .git; проверенные head и merge-base уже присутствуют локально и совпадают с данными API. Код не изменялся.

Повторное независимое ревью PR #58, head 32e26ccdd2b691ca6f04f49bbde0554df2f287af: замечаний, требующих исправления, не обнаружено. PR уже слит; существующее APPROVED для этого SHA сохраняется. Подтверждены совпадение origin с admin/miem_workers и независимость Gitea-аккаунта agent (id 7) от автора PR codex (id 6). Изучены все 9 изменённых файлов и связанный поток route → council_candidates → template, стили и footer. Проверены разрешённые параметры сортировки, значения по умолчанию, стабильная сортировка по вторичным ключам, применение до пагинации и сохранение выбора при поиске, сбросе и переходах. Версии согласованы на 0.12.0. Изучены unit- и browser-сценарии. git diff --check прошёл. Через API подтверждён success для CI / test на проверенном SHA; workflow включает pytest и синтаксические проверки. Локальные тесты и браузер не запускались в рамках ограничений reviewer-сессии. Fetch недоступен из-за read-only .git; проверенные head и merge-base уже присутствуют локально и совпадают с данными API. Код не изменялся.
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: admin/miem_workers#58