Repository navigation
prod: компактная команда, детализация кейсов и lifecycle проектов - #391
Merged
Merged
Conversation
13 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Что изменено
Выборочный production-перенос пяти принятых на DEV PR: #386 Team, #387 case drilldown, #388 lifecycle priority, #389 Windows-only Vitest и #390 удаление общей кнопки Cases. Остальные изменения
devне переносились.Мержить только после backend #761 → Release Prod → успешного API smoke. Push в Angular master автоматически запускает production deploy.
Зачем
DEV acceptance завершён пользователем. Подготовить отдельный PR в
master, сохранив production architecture и несвязанные flows.Реализация
Чистая база после fetch:
ba79d200e678c4a2c1e814dfe9ca93113bcfbf25.case_scope=allи прочие Analytics exports сохранены.submitted > program > draft > published; draft с программой показывает «В программе». Permissions/API/HTML/SCSS карточек не менялись.deps.inlineтолько приprocess.platform === "win32"; из production config удалён прежнийpool: "forks". Linux использует обычную конфигурацию, командаnpm run test:ciбез дополнительных флагов.Production
/project-analytics/, attention URLs и подписи соседних метрик сохранены; DEV/manager-overview/не переносился. Package/lock, coverage/exclusions и production workflows не менялись.API
Case list/export используют совместимое opt-in расширение из backend production PR #761. Backend выпускается первым. Презентация —
Project.presentation_address; export полного кейса не зависит от search/page. API contracts команды/карточек не менялись.UI
Сохранены утверждённый DEV visual world, Mont и production patterns. Нет sidebar/promo/старой invite-плашки в Team; нет «Кейс»/evaluation колонок в case drilldown; нет общей кнопки Cases.
Browser QA на реальных production Angular-компонентах с локальными fixtures:
Browser errors 0; side-by-side с принятым DEV-вариантом — PASS. Полный screen-reader audit не заявляется.
Permissions
Manager UI gate, Team permissions и правила доступа карточек сохранены. Staff/superuser UI access не расширяется; backend авторизует каждый запрос.
Проверки
npm run lint:ts(0 errors, 6 существующих warnings), Stylelint изменённых SCSS,npm run build:prod,git diff --check— PASS.npm run test:ci, без дополнительных pool/worker flags.npm run format:check,npm run lint:ts,npm run build:prod— PASS.CI run 36488758594 проверил точный HEAD e49d48f. Использована существующая диагностическая ветка с явным checkout, штатными проверками и production build; deploy пропущен. В production workflows нет PR-trigger; диагностические изменения в этот PR не входят.
Review по
procollab-review: BLOCKER/MAJOR/MINOR в итоговом selective diff не выявлены. Во время переноса устранены различия module names/URLs и сохранена production geometry общего invite dialog. Review не требует изменений принятых product decisions.Acceptance criteria
Не проверено
Live production/API integration после release — smoke выполняет пользователь. Browser QA использовал fixture repository; настоящие данные/permissions дополнительно проверяются backend tests и production smoke.
Риски / migration
Backend production должен быть выпущен и проверен раньше merge Angular. Push в Angular
masterавтоматически запускает штатный production deploy. Миграции не нужны.Rollback: откатить этот frontend commit штатным процессом; новый backend совместим со старым Angular. Merge/release/deploy Codex не выполнял.