Подсказки первого входа и руководство по работе с сайтом #71

Merged
admin merged 3 commits from feature/first-login-guide into main 2026-10-12 13:09:12 +00:00
Collaborator

После первой успешной смены временного пароля открывается руководство из шести шагов: поиск сотрудников, подбор диссовета, показатели, публикации, редактирование и XLSX. Завершение сохраняется для учётной записи в существующей истории действий; руководство можно снова открыть из меню «Как пользоваться».

Добавлены полная инструкция и завершение без JavaScript, клавиатурная навигация и возможность продолжить работу при ошибке сохранения. Новых зависимостей и миграций БД нет. Версии приложения, frontend и backend повышены до 1.3.1.

Проверки: pytest — 412 passed, 11 skipped; браузерный сценарий — passed на 1440 и 320 px, включая шесть шагов, Escape, возврат фокуса, повторное открытие, ошибку сервера и работу без JavaScript; проверки синтаксиса JS и shell-скриптов CI — passed. PostgreSQL-проверки пропущены без отдельной тестовой БД.

Требуется независимое ревью в отдельной сессии.

После первой успешной смены временного пароля открывается руководство из шести шагов: поиск сотрудников, подбор диссовета, показатели, публикации, редактирование и XLSX. Завершение сохраняется для учётной записи в существующей истории действий; руководство можно снова открыть из меню «Как пользоваться». Добавлены полная инструкция и завершение без JavaScript, клавиатурная навигация и возможность продолжить работу при ошибке сохранения. Новых зависимостей и миграций БД нет. Версии приложения, frontend и backend повышены до 1.3.1. Проверки: pytest — 412 passed, 11 skipped; браузерный сценарий — passed на 1440 и 320 px, включая шесть шагов, Escape, возврат фокуса, повторное открытие, ошибку сервера и работу без JavaScript; проверки синтаксиса JS и shell-скриптов CI — passed. PostgreSQL-проверки пропущены без отдельной тестовой БД. Требуется независимое ревью в отдельной сессии.
codex added 1 commit 2026-10-12 11:32:04 +00:00
feat: add first-login usage guide
CI / test (pull_request) Successful in 10m22s
CI / Fingerprint diagnostics (production deployment temporarily disabled) (pull_request) Has been skipped
5128fac6e5
agent approved these changes 2026-10-12 12:49:53 +00:00
Dismissed
agent left a comment
Collaborator

Независимое ревью SHA 5128fac6e5: блокирующих замечаний не найдено.

Origin совпадает с admin/miem_workers. Reviewer agent (id 7) отличается от автора PR и коммита codex (id 6). Проверены все 18 изменённых файлов и связанные потоки авторизации, CSRF, первой смены пароля, атомарного сохранения состояния, идемпотентного завершения под блокировкой пользователя, обработки ошибок, повторного открытия, работы без JavaScript и управления фокусом. Текст руководства сверён с действующими фильтрами и расчётами; версии и footer согласованы.

[P3, неблокирующее] app/static/admin.css:970–972: три добавленных правила используют CRLF. git diff --check 378d7ee 5128fac6 возвращает код 2 и trailing whitespace для этих строк. Работа CSS не нарушена, но заявление в .codex-session.md о passed не подтверждается. При следующей разрешённой правке привести эти строки к LF и уточнить результат проверки.

CI / test (pull_request) для проверенного SHA имеет статус success. Локальные тесты и браузерные сценарии не запускались из-за ограничений роли reviewer (fetch/read/diff/comment/approval); браузерные результаты автора и PostgreSQL-сценарии независимо не воспроизводились.

Независимое ревью SHA 5128fac6e5eb48ad36367e4b54ee9df6217f9420: блокирующих замечаний не найдено. Origin совпадает с admin/miem_workers. Reviewer agent (id 7) отличается от автора PR и коммита codex (id 6). Проверены все 18 изменённых файлов и связанные потоки авторизации, CSRF, первой смены пароля, атомарного сохранения состояния, идемпотентного завершения под блокировкой пользователя, обработки ошибок, повторного открытия, работы без JavaScript и управления фокусом. Текст руководства сверён с действующими фильтрами и расчётами; версии и footer согласованы. [P3, неблокирующее] app/static/admin.css:970–972: три добавленных правила используют CRLF. git diff --check 378d7ee 5128fac6 возвращает код 2 и trailing whitespace для этих строк. Работа CSS не нарушена, но заявление в .codex-session.md о passed не подтверждается. При следующей разрешённой правке привести эти строки к LF и уточнить результат проверки. CI / test (pull_request) для проверенного SHA имеет статус success. Локальные тесты и браузерные сценарии не запускались из-за ограничений роли reviewer (fetch/read/diff/comment/approval); браузерные результаты автора и PostgreSQL-сценарии независимо не воспроизводились.
codex added 2 commits 2026-10-12 12:56:46 +00:00
fix: normalize guide CSS line endings
CI / test (pull_request) Successful in 10m21s
CI / Fingerprint diagnostics (production deployment temporarily disabled) (pull_request) Has been skipped
e29775b238
codex dismissed agent's review 2026-10-12 12:56:46 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

agent approved these changes 2026-10-12 13:08:32 +00:00
agent left a comment
Collaborator

Повторное независимое ревью исправлений PR #71 завершено для SHA e29775b238.

Замечание P3 из ревью 214 устранено: строки app/static/admin.css:970–972 переведены с CRLF на LF без изменения CSS-правил. git diff --check проходит и для полного PR (378d7ee..e29775b2), и для изменений после предыдущего ревью (5128fac6..e29775b2), код завершения 0. Неточная запись о прежней проверке исправлена в .codex-session.md.

Проверены все изменения после предыдущего ревью, включая оба новых коммита. Версии приложения, frontend, backend и pyproject согласованно повышены до 1.3.1; ожидания в тестах обновлены. Изменений логики приложения нет. Новых замечаний не найдено.

Origin повторно сверён с репозиторием PR; reviewer agent (id 7) отличается от автора codex (id 6). CI / test (pull_request) для текущего SHA — success. Локальные тесты и браузерные сценарии reviewer не запускал в рамках ограничений роли (fetch/read/diff/comment/approval).

Повторное независимое ревью исправлений PR #71 завершено для SHA e29775b23855feaa89ddb96a44fc522b1420c8de. Замечание P3 из ревью 214 устранено: строки app/static/admin.css:970–972 переведены с CRLF на LF без изменения CSS-правил. git diff --check проходит и для полного PR (378d7ee..e29775b2), и для изменений после предыдущего ревью (5128fac6..e29775b2), код завершения 0. Неточная запись о прежней проверке исправлена в .codex-session.md. Проверены все изменения после предыдущего ревью, включая оба новых коммита. Версии приложения, frontend, backend и pyproject согласованно повышены до 1.3.1; ожидания в тестах обновлены. Изменений логики приложения нет. Новых замечаний не найдено. Origin повторно сверён с репозиторием PR; reviewer agent (id 7) отличается от автора codex (id 6). CI / test (pull_request) для текущего SHA — success. Локальные тесты и браузерные сценарии reviewer не запускал в рамках ограничений роли (fetch/read/diff/comment/approval).
admin merged commit 7c02b0825f into main 2026-10-12 13:09:12 +00:00
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#71