PROD: обновить список назначений экспертов - #356
Merged
Merged
Conversation
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.
Селективный перенос DEV #354 и #355 на актуальный PROD master. В назначениях экспертов удалены колонка «Прогресс», поля критериев и
assignmentProgress(). Все три scope используют существующий общий шаблон с проверенным DEV-оформлением: пропорции колонок 27/31/29/13%, отдельный status badge и действие «Посмотреть оценку», мобильные карточки до 1000 px.PROD-пути
/programs/:programId/project-analytics/...сохранены. Модификаторы body/table включаются только приview === "assignments"; backlog, scores detail и таблицы внимания не получают новое оформление. Статусы, ожидание, «Завершили: X из Y», API handlers, permissions, router и sharedapp-modalне изменены.Точные SHA и источники
d287b9196d0fb186da050a331cb8dcadb99b056e58969092763da3ffcb22b44f8533475b0d431e7642cbdd65117cc56beaa6069bcc061192df7733ce5b430e7e3a9bec14240bc02528068248feb6494b5a0ad3f28d5a26c935e8caf110add33507f316f4Перенесены только относящиеся к задаче изменения; merge dev → master и cherry-pick DEV merge commits не выполнялись. HTML/SCSS/TS компонента совпадают с проверенным DEV-компонентом. Старые DEV API-пути и посторонние документы не переносились.
Изменённые файлы
projects/core/src/lib/interceptors/camelcase.interceptor.spec.tsprojects/social_platform/src/app/domain/program/program-analytics-assignment.fixture.tsprojects/social_platform/src/app/domain/program/program-analytics.model.tsprojects/social_platform/src/app/ui/pages/program/detail/analytics/drilldown/analytics-drilldown.component.htmlprojects/social_platform/src/app/ui/pages/program/detail/analytics/drilldown/analytics-drilldown.component.scssprojects/social_platform/src/app/ui/pages/program/detail/analytics/drilldown/analytics-drilldown.component.tsprojects/social_platform/src/app/ui/pages/program/detail/analytics/drilldown/analytics-drilldown.component.spec.tsprojects/social_platform/src/app/utils/program-analytics-assignment.tsprojects/social_platform/src/app/utils/program-analytics-assignment.spec.tsdocs/modules/program.mdПроверки
Node 20.20.2, все команды exit 0:
npm cinpm run format:check; после уточнения документации дополнительно scoped Prettier.npm run lint:ts— 0 errors, 6 существующих warnings.analytics-drilldown.component.scss.npx vitest run program-analytics analytics-drilldown analytics-attention manager-analytics manager-assignments manager-assignment-scores camelcase.interceptor— 141/141, 9 файлов.npm run test:ci— 1756/1756, 377 файлов. Без NG0401 и unhandled errors. Использован уже существующий в masterpool: "forks"; test config не менялся.npm run build:prod— успешно, существующие warnings сборки.git diff --check— успешно.Локальный браузерный smoke на настоящем Angular-компоненте с синтетическими данными: all/completed/pending, ровно четыре колонки, все статусы, scores/back, Escape и возврат фокуса; backlog и attention без assignment-модификаторов. Desktop 1280 × 900: окно 980 px, колонки 249,47 / 286,44 / 267,95 / 120,14 px, строки 72–77 px. На ширинах 320, 390, 768, 999, 1000 и 1280 px горизонтального overflow нет; длинные имена и названия переносятся, mobile status/action разделены. Console errors/warnings отсутствуют. Авторизованный smoke на PROD не выполнялся.
Границы и будущий релиз
React untouched; DEV untouched; workflows untouched. Зависимости, lockfile, Docker, test config, backend, current application и shared app-modal не изменялись.
Связанный backend Draft PR #747, head
02abcec3eda64eb7e277e1af19003e70d18e1feb; источник api #746.После отдельного подтверждения пользователя: сначала Angular merge и успешный frontend PROD deploy, затем проверка назначений; только потом backend merge и Release Prod exact backend merge SHA. Новый Angular совместим со старым backend, дополнительные поля игнорируются. Backend первым выкатывать нельзя.
Этот PR остаётся Draft. Merge/deploy не выполнялись.